Closed
Bug 1398264
Opened 7 years ago
Closed 7 years ago
Private browsing icon missing from private windows on Nightly
Categories
(Firefox :: Private Browsing, defect)
Tracking
()
VERIFIED
FIXED
Firefox 57
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox55 | --- | unaffected |
firefox56 | --- | unaffected |
firefox57 | --- | verified |
People
(Reporter: haik, Assigned: Gijs)
References
Details
(Keywords: regression)
Attachments
(3 files)
On OS X, the private browsing icon is missing from the top-right of the title bar on today's Nightly. mozregression: 5:39.23 INFO: No more inbound revisions, bisection finished. 5:39.23 INFO: Last good revision: 481d7cc2f6f06ea35794403dc9291d277910d6b9 5:39.23 INFO: First bad revision: 1fae43b4534cf25d917e544ca02e7ad6b1aa1736 5:39.23 INFO: Pushlog: https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=481d7cc2f6f06ea35794403dc9291d277910d6b9&tochange=1fae43b4534cf25d917e544ca02e7ad6b1aa1736 Which is bug 1397236 - "Changing positioning of navigator-toolbox while a descendent popup is open causes the popup to close. We don't need to change the positioning of the navigator-toolbox on the fly, and can instead always keep it as position:relative for the library animation."
Reporter | ||
Updated•7 years ago
|
Keywords: regression
Comment 1•7 years ago
|
||
Drew, do you think you could help take a look at this since I'm going to be away for the next two days?
Flags: needinfo?(jaws) → needinfo?(adw)
Assignee | ||
Comment 2•7 years ago
|
||
I saw bugmail for this and figured it out pretty quickly, so I wrote a patch.
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Flags: needinfo?(adw)
Comment hidden (mozreview-request) |
Comment 4•7 years ago
|
||
(In reply to Haik Aftandilian [:haik] from comment #0) > On OS X, ... and on Windows. Gijs, your patch doesn't seem to fix this regression on windows?
Assignee | ||
Comment 5•7 years ago
|
||
(In reply to Ekanan Ketunuti from comment #4) > (In reply to Haik Aftandilian [:haik] from comment #0) > > On OS X, ... > > and on Windows. > > Gijs, your patch doesn't seem to fix this regression on windows? No, because nobody said that there was an issue on Windows until you did, and the element in question is osx-specific ( https://dxr.mozilla.org/mozilla-central/source/browser/base/content/browser.xul#605-610 ), and Jared normally works on Windows, so it seemed logical to assume the issue was restricted to OS X. I'll check Windows and see what's the matter there.
Assignee | ||
Updated•7 years ago
|
OS: Unspecified → All
Hardware: Unspecified → All
Assignee | ||
Comment 6•7 years ago
|
||
I actually can't reproduce this on windows 10 with the Sept. 10 nightly, no matter whether the title bar is on/off, whether I'm using normal, touch or compact, density, or a lightweight theme or the default theme. Can you clarify where you're seeing this on Windows?
Flags: needinfo?(ananuti)
Comment 7•7 years ago
|
||
(In reply to :Gijs from comment #6) > I actually can't reproduce this on windows 10 with the Sept. 10 nightly, no > matter whether the title bar is on/off, whether I'm using normal, touch or > compact, density, or a lightweight theme or the default theme. Can you > clarify where you're seeing this on Windows? Hmm, interesting. It looks like I can reproduce only with dark theme. just enable dark theme and press Ctrl-Shift-P.
Flags: needinfo?(ananuti)
Comment 8•7 years ago
|
||
(In reply to Ekanan Ketunuti from comment #7) > Hmm, interesting. It looks like I can reproduce only with dark theme. I cannot reproduce with Default theme but Dark and Light theme.
Comment 9•7 years ago
|
||
Oh, pressing Alt button and I found PB icon is on munubar.
Comment hidden (mozreview-request) |
Assignee | ||
Comment 12•7 years ago
|
||
FWIW, I checked, and I don't think this happens on Linux because we don't have tabsintitlebar there at all.
Comment 13•7 years ago
|
||
mozreview-review |
Comment on attachment 8906559 [details] Bug 1398264 - position private browsing indicator and/or 10.9 fullscreen button to cope with the background on the toolbox, https://reviewboard.mozilla.org/r/178308/#review183394 Thanks
Attachment #8906559 -
Flags: review?(adw) → review+
Comment 14•7 years ago
|
||
Pushed by gijskruitbosch@gmail.com: https://hg.mozilla.org/integration/autoland/rev/cbd4482d8d2e position private browsing indicator and/or 10.9 fullscreen button to cope with the background on the toolbox, r=adw
Comment 15•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/cbd4482d8d2e
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Updated•7 years ago
|
status-firefox55:
--- → unaffected
status-firefox56:
--- → unaffected
status-firefox-esr52:
--- → unaffected
Comment 17•7 years ago
|
||
I have reproduced this bug with Nightly 57.0a1 (2017-09-08) on Windows 10 , 64 Bit ! This bug's fix is Verified with latest Beta ! Build ID 20171005195903 User Agent Mozilla/5.0 (Windows NT 10.0; WOW64; rv:57.0) Gecko/20100101 Firefox/57.0
QA Whiteboard: [bugday-20171004]
Updated•6 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•