Closed
Bug 818732
Opened 12 years ago
Closed 11 years ago
Switch Nightly to per-window private browsing
Categories
(Firefox :: Private Browsing, defect)
Tracking
()
RESOLVED
FIXED
Firefox 20
People
(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)
References
Details
Attachments
(1 file)
481 bytes,
patch
|
jdm
:
review+
|
Details | Diff | Splinter Review |
We're very close to a point where we can switch Nightly to per-window private browsing by default. I'm hoping to do that some time early next week.
Assignee | ||
Updated•12 years ago
|
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #689004 -
Flags: review?(josh)
Updated•12 years ago
|
Attachment #689004 -
Flags: review?(josh) → review+
Comment 2•12 years ago
|
||
Should we do away with the private browsing service here as well?
Assignee | ||
Comment 3•12 years ago
|
||
(In reply to comment #2) > Should we do away with the private browsing service here as well? We still need a bit more work for that to happen... I'll do some of that work tonight. I'll file a separate bug when I'm ready to remove the PB service, but that doesn't necessarily block us to enable this on Nightly.
Assignee | ||
Comment 4•12 years ago
|
||
(This is the first chunk of work that I was talking about: https://hg.mozilla.org/mozilla-central/pushloghtml?changeset=7dcc0be8d4aa)
Assignee | ||
Comment 5•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/18baa7203214
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 20
Assignee | ||
Comment 6•12 years ago
|
||
Backed out because I had to back out bug 769285: https://hg.mozilla.org/mozilla-central/rev/1fe0626d17c3
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 7•11 years ago
|
||
Talos seems to be faily happy with this: <http://perf.snarkfest.net/compare-talos/index.html?oldRevs=99f219056c3e&newRev=035274d07e19&tests=a11y,tdhtml,tdhtml_nochrome,a11y_paint,tdhtml_paint,tdhtml_nochrome_paint,tp4,tp4_memset,tp4_pbytes,tp4_rss,tp4_shutdown,tp4_xres,tp4m,tp4m_content_rss,tp4m_main_rss,tp4m_main_rss_nochrome,tp4m_nochrome,tp4m_shutdown,tp4m_shutdown_nochrome,tp5r,tp5r_memset,tp5r_pbytes,tp5r_rss,tp5r_shutdown,tp5r_xres,tp5r_paint,tp5r_memset_paint,tp5r_pbytes_paint,tp5r_%cpu_paint,tp5r_modlistbytes_paint,tp5r_rss_paint,tp5r_shutdown_paint,tp5r_xres_paint,dromaeo_basics,dromaeo_css,dromaeo_dom,dromaeo_jslib,dromaeo_sunspider,dromaeo_v8,tsspider,tsspider_nochrome,tsspider_paint,tsspider_nochrome_paint,v8,tgfx,tgfx_nochrome,tgfx_paint,tgfx_nochrome_paint,tscroll,tsvg,tsvg_opacity,tzoom,ts,ts_paint,ts_cold,ts_cold_generated_max,ts_cold_generated_max_shutdown,ts_cold_generated_med,ts_cold_generated_med_shutdown,ts_cold_generated_min,ts_cold_generated_min_shutdown,ts_cold_shutdown,ts_places_generated_max,ts_places_generated_max_shutdown,ts_places_generated_med,ts_places_generated_med_shutdown,ts_places_generated_min,ts_places_generated_min_shutdown,ts_shutdown,twinopen,tpaint&submit=true>
Assignee | ||
Comment 8•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/020555b69f72
Status: REOPENED → RESOLVED
Closed: 12 years ago → 11 years ago
Resolution: --- → FIXED
Comment 9•11 years ago
|
||
Backed out for causing bug 821701: https://hg.mozilla.org/integration/mozilla-inbound/rev/8286fd779a20
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 10•11 years ago
|
||
(In reply to Ed Morley [UTC+0; email:edmorley@moco] from comment #9) > Backed out for causing bug 821701: > https://hg.mozilla.org/integration/mozilla-inbound/rev/8286fd779a20 Merge of backout: https://hg.mozilla.org/mozilla-central/rev/8286fd779a20
Assignee | ||
Comment 11•11 years ago
|
||
Relanded after disabling the crashing test: https://hg.mozilla.org/mozilla-central/rev/2798371f1650
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•