Closed Bug 771188 Opened 8 years ago Closed 6 years ago
Provide the ability to enter private browsing from within Firefox Metro
Firefox Metro currently does not have private browsing. This task is to add an option and keyboard shortcut to enter private browsing when in Metro XUL Fennec. The Metro browser should simply be restarted in private browsing mode for now. I couldn't find an option in IE10 to enter private browsing when in Metro mode, but I could find an option to do this in Metro Chrome.
Why would we want to restart Firefox for this?
I figured simply restarting the browser with firefox.exe -private would be a straightforward and fast implementation, but if you have a better suggestion please share. I'm not familiar with private browsing code yet, but I know you are :)
I think we want to get per-window PB working here first, and then just set the flag on the docshell.
Depends on: PBnGen
There is only 1 Window in Metro mode.
(In reply to comment #4) > There is only 1 Window in Metro mode. Yes. What I meant was that it would be way easier to implement this on top of the infrastructure that we're working on for per-window PB.
Is there enough of that new infrastructure landed already that I (or someone else) can work on this? If not, do you know when that will be ready?
(In reply to comment #6) > Is there enough of that new infrastructure landed already that I (or someone > else) can work on this? If not, do you know when that will be ready? Not yet. The timeline is not clear. We currently don't have anybody working on this, and are getting contributions from the community. Chris Lord wanted to work on this as a prerequisite for the mobile PB mode, but I think he's now pulled off into other projects. :(
Are you opposed to me filing another bug to do the post new infrastructure work in a new bug, and using this one to restart the browser in PB mode?
(In reply to comment #8) > Are you opposed to me filing another bug to do the post new infrastructure work > in a new bug, and using this one to restart the browser in PB mode? Yes, I don't think that's the direction that we want to take. I'm fine with you landing your changes here hidden behind MOZ_PER_WINDOW_PRIVATE_BROWSING which is not defined yet. Not sure how much that helps you. What's the timeline for this work from a Firefox Metro perspective?
(In reply to Ehsan Akhgari [:ehsan] from comment #7) > (In reply to comment #6) > > Is there enough of that new infrastructure landed already that I (or someone > > else) can work on this? If not, do you know when that will be ready? > > Not yet. The timeline is not clear. We currently don't have anybody > working on this, and are getting contributions from the community. Chris > Lord wanted to work on this as a prerequisite for the mobile PB mode, but I > think he's now pulled off into other projects. :( I must apologise for jumping the gun a bit, I assumed that my work on bug 758620 would be over pretty quickly, but the problems it caused/exposed took a while to sort out. I'm hoping that with bug 769541 looking like it's finally fixed, I'll actually be able to start work on this properly, but I'll be away for a week and a half from Wednesday, so I won't seriously be able to start before then. Until then, don't let me block anything, but I really ought to be able to start working on this properly come August 6th.
Personally, I'm inclined to let Brian implement PB mode however he wants to at the moment. I'm not convinced per-window will make it in time for 17.
> What's the timeline for this work from a Firefox Metro perspective? There's no detailed requirements. I think Windows 8 should be released at the end of October. The goals list have alpha/beta by the end of the year. https://wiki.mozilla.org/Firefox/Roadmap > I'm fine with you landing your changes here hidden behind > MOZ_PER_WINDOW_PRIVATE_BROWSING which is not defined yet. I'd rather wait in that case.
According to https://wiki.mozilla.org/RapidRelease/Calendar, shipping something in beta by the end of the year corresponds to Firefox 18, so we still have a bit of time. But if/when this becomes a priority, please check back with me, as we might need to get somebody working on this full-time (assuming that Chris would get pulled into other projects...) Sorry for being so strict here, but I don't want us to resort to a hack (restarting the browser) until we absolutely have to... :/
np! Thanks for the info.
Would we need multi window support in order to consume the new work being done for private browsing per window? Can we do it per tab easily?
(In reply to comment #15) > Would we need multi window support in order to consume the new work being done > for private browsing per window? Can we do it per tab easily? The new API is per-docshell. While Firefox Desktop would not have the UI for this, it is possible to get it to work per-tab.
Component: Private Browsing → General
Product: Firefox → Firefox for Metro
Version: unspecified → Trunk
This can be implemented now right Ehsan?
(In reply to comment #18) > This can be implemented now right Ehsan? Yeah, but I'd like us to sit down some time and have a chat about this so that everybody knows where we're going and what we need to do.
We won't block shipping Metro on private browsing. We can get to this in a v2. For v1, if a user needs private browsing, she can use the desktop version.
Whiteboard: [metro-mvp] [LOE:2] → [metro-mvp-] [LOE:2]
For scoping/estimating purposes, front-end work will include: * Creating a private tab * Displaying private tabs in the tab strip * Custom "new tab" experience for private tabs * Inheriting private status when opening links/searches in new tabs * Preserving (or not?) state when switching between desktop and Metro
Here's a patch that adds a "new private tab" button in the Metro tab strip. This is nowhere close to the final UX, and it is missing some vital privacy pieces like integration with sessionstore. However, it could be useful if you want to test how the platform feature works when running in the Metro UI.
Note that private tabs do not work 100% correctly. We have been operating under the assumption that once a docshell is private, all of its child docshells are private.
Here's a slightly more complete patch, which adds proper session store support, a keyboard shortcut, makes new tabs inherit the private state from the parent tab if there is one, and adds a pref to hide the (incomplete) UI by default.
Attachment #8356806 - Attachment is obsolete: true
Comment on attachment 8358579 [details] [diff] [review] WIP I moved this patch to bug 962212, and filed some additional bugs for remaining work (see this bug's dependency list).
Attachment #8358579 - Attachment is obsolete: true
7 years ago
Depends on: 963678
moving [story] bugs over to tracking product.
Component: Components → Metro Operations
OS: Windows 8 Metro → Windows 8.1
Product: Firefox for Metro → Tracking
Version: Trunk → ---
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.