Opening PDF in a tab shows chrome://extension URL and PDF extension favicon
chrome-extension://oemmndcbldboiebfnladdacbdfmadadm/ to URL## Actual result:

Should not show the extension prefix and should load the website favicon. Only show extension icon if site doesn't have favicon set
Install build
I think this is actually OK for now, so moving to backlog. Valid though.
Came here to report this; since there's already an open issue, I'll just add a comment to it. It looks like this is being worked on, so hopefully this will soon be moot, but I just want to respond to @bbondy's "it's OK" comment.
As an academic, I spend a lot of time reading PDF documents. It's not uncommon for me to go to a Web site (say a professor's publication list) and open up a bunch of documents in new tabs, and start reading. Eventually, I might get to one that's really interesting and I want to share with others. I can't just copy the URL and send it to someone else; either I have to manually edit it or they have to. Many people I know lack the facility to do this on either the sending or receiving side.
Since this is not the behavior in other browsers, there's no reason people would "just know" how to do such things. Especially since, to most users, the prefix (chrome-extension://oemmndcbldboiebfnladdacbdfmadadm/) looks like gibberish, and hence intimidating. Worse, in a context where the URL is presented through just a truncated prefix (e.g., a tweet), they wouldn't even _see_ the suffix where the "real" URL resides.
In short, I think this is pretty user-unfriendly, and as Brave starts getting more non-expert users, this is bound to cause confusion and consternation.
Thanks for working on fixing this.
This problem also hides the site info popup, which makes it harder to find the HTTPS certificate, cookies, and site settings.
Normal:

PDF:

I think we can close this in favor of #3846.
yep
Most helpful comment
Came here to report this; since there's already an open issue, I'll just add a comment to it. It looks like this is being worked on, so hopefully this will soon be moot, but I just want to respond to @bbondy's "it's OK" comment.
As an academic, I spend a lot of time reading PDF documents. It's not uncommon for me to go to a Web site (say a professor's publication list) and open up a bunch of documents in new tabs, and start reading. Eventually, I might get to one that's really interesting and I want to share with others. I can't just copy the URL and send it to someone else; either I have to manually edit it or they have to. Many people I know lack the facility to do this on either the sending or receiving side.
Since this is not the behavior in other browsers, there's no reason people would "just know" how to do such things. Especially since, to most users, the prefix (
chrome-extension://oemmndcbldboiebfnladdacbdfmadadm/) looks like gibberish, and hence intimidating. Worse, in a context where the URL is presented through just a truncated prefix (e.g., a tweet), they wouldn't even _see_ the suffix where the "real" URL resides.In short, I think this is pretty user-unfriendly, and as Brave starts getting more non-expert users, this is bound to cause confusion and consternation.
Thanks for working on fixing this.