Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

about: pages fail load in private tabs #1817

Closed
diracdeltas opened this issue May 20, 2016 · 9 comments
Closed

about: pages fail load in private tabs #1817

diracdeltas opened this issue May 20, 2016 · 9 comments

Comments

@diracdeltas
Copy link
Member

diracdeltas commented May 20, 2016

Test plan

  1. Launch Brave and open a new private tab
  2. The new tab should be blank (empty white page)
  3. Navigate to about:preferences and confirm that it loads properly
  4. Navigate to about:brave and confirm that it loads properly

Original issue description

started happening today or yesterday

@diracdeltas diracdeltas added this to the 0.10.0dev milestone May 20, 2016
@diracdeltas diracdeltas changed the title opening a new tab opens about:error instead of about:newtab opening a new private tab opens about:error instead of about:newtab May 20, 2016
@diracdeltas
Copy link
Member Author

actually looks like any about: page in a private tab will just redirect to about:error. @bridiver thoughts?

@bridiver
Copy link
Collaborator

opening a private tab triggers a did-fail-load event with for addressUnreachable (-109). Not sure why

@bridiver
Copy link
Collaborator

it's really only failing the provision load, but electron fires did-fail-load and did-fail-provisional-load in that method. I'm not sure why it only happens with private tabs, but I have a workaround until we can figure it out.

@bridiver
Copy link
Collaborator

maybe not. The workaround still doesn't actually load the page for about:preferences and other non-blank pages

@bridiver bridiver reopened this May 20, 2016
@bridiver
Copy link
Collaborator

This is actually not a new issue, we just never noticed that the page didn't actually load. Trying to view about:preferences in a private tab results in a blank page. It probably doesn't make sense to view about pages (other than newtab) in a private session because they don't have their own set of persistent preferences, bookmarks, etc... The workaround I have will at least show newtab instead of error, but I think we should open other about:* pages in a new tab if the url is manually entered into a private tab. about:certerror will also display a blank error page in a private tab.

bridiver added a commit that referenced this issue May 20, 2016
@bbondy bbondy modified the milestones: 0.10.1dev, 0.10.0dev May 20, 2016
@bbondy
Copy link
Member

bbondy commented May 20, 2016

Moving t 0.10.1 since there's as workaround

@diracdeltas
Copy link
Member Author

The workaround I have will at least show newtab instead of error, but I think we should open other about:* pages in a new tab if the url is manually entered into a private tab. about:certerror will also display a blank error page in a private tab.

Thanks @bridiver. I believe at least the following pages should be displayable in a private tab:

  • about:blank (needed for X-XSS-Protection block mode, and maybe some site functionality)
  • about:newtab
  • about:certerror
  • about:error
  • about: safebrowsing

In FF, all the about:about pages are displayable in a private window. In Chrome, at least settings, history, and bookmarks (but not downloads) open in a regular window if you try to open them in incognito mode.

bridiver added a commit that referenced this issue May 20, 2016
bridiver added a commit that referenced this issue May 20, 2016
@diracdeltas diracdeltas changed the title opening a new private tab opens about:error instead of about:newtab about: pages fail load in private tabs May 20, 2016
@bbondy bbondy modified the milestones: 0.10.2dev, 0.10.1dev May 30, 2016
@bbondy
Copy link
Member

bbondy commented Jun 15, 2016

Moving t 0.10.5 for now.

@bbondy bbondy modified the milestones: 0.10.5dev, 0.10.4dev, 0.11.0dev Jun 15, 2016
@bbondy bbondy modified the milestones: 0.11.2dev, 0.11.1dev Jul 12, 2016
@bbondy bbondy removed this from the 0.11.2dev milestone Jul 21, 2016
@bbondy
Copy link
Member

bbondy commented Mar 28, 2017

cc51723

Was due to incognito manifest setting of spanning not allowing main frame url loads.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants