Closed Bug 1228032 Opened 6 years ago Closed 6 years ago

Toolbox opened in separate window couldn't be resized, moved and closed

Categories

(DevTools :: General, defect)

defect
Not set
normal

Tracking

(firefox45+ verified)

VERIFIED FIXED
Firefox 45
Tracking Status
firefox45 + verified

People

(Reporter: arni2033, Assigned: Gijs)

References

Details

(Keywords: regression)

Attachments

(1 file)

>>>   My Info:   Win7_64, Nightly 45, 32bit, ID 20151125030231
STR:
1. Open devtools (Ctrl+Shift+I)
2. Click "Show in separate window" button

Result:       
 Toolbox opens in separate window, but there's no close and resize buttons, no title,
 and there's no way to resize, move or close it.

Expectations: 
 The window should look like it used to, or at least: there should be a way to view window title,
 to resize, move and close the window.

It was regressed between 2015-11-23 and 2015-11-24, probably by bug 1226570:
> pushlog_url:   https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=856cffd3910bcb90c681f91ba0acfee9088722c1&tochange=20169c82ed443f7158cf2143223c858647ffe417
[Tracking Requested - why for this release]: Not sure I'll get to this before PTO, but we should definitely take a look at this.
If I open browser toolbox, which is also broken, open frame "toolbox-window.xul" then create a new stylesheet with CSS from chrome://browser/skin/browser.css - then everything's OK.
Looking at the changes made in bug 1226570, probably the goal was to _replace_ stylesheets link, not remove them completely? Currently toolbox window looks as if "-moz-appearance:none" was applied
I'm pretty sure you want to have chrome://global/skin/[global.css] as a stylesheet for all of these, or you will have a Bad Time.
I can't repro this on mac. Is it windows only?
Bug 1228032 - restore chrome://global/skin/ where chrome://browser/skin/ was removed, r?jryans
Attachment #8692187 - Flags: review?(jryans)
Comment on attachment 8692187 [details]
MozReview Request: Bug 1228032 - restore chrome://global/skin/ where chrome://browser/skin/ was removed, r?jryans

https://reviewboard.mozilla.org/r/26233/#review23643

Okay, since you've confirmed this fixes the issue, we can go with this.

I **believe** we only actually need this in the `toolbox-*-window.xul` files, though.
Attachment #8692187 - Flags: review?(jryans) → review+
Tree's closed, and I need to go sleep.

(In reply to J. Ryan Stinnett [:jryans] (use ni?) (on PTO until Dec. 7) from comment #6)
> Comment on attachment 8692187 [details]
> MozReview Request: Bug 1228032 - restore chrome://global/skin/ where
> chrome://browser/skin/ was removed, r?jryans
> 
> https://reviewboard.mozilla.org/r/26233/#review23643
> 
> Okay, since you've confirmed this fixes the issue, we can go with this.
> 
> I **believe** we only actually need this in the `toolbox-*-window.xul`
> files, though.

I believe the same thing, but I'd rather find out after fixing this for the upcoming nightly. Of course, with the tree closed it's difficult to make sure that happens...
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/5437ff201b3a
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 45
See Also: 1228049
Duplicate of this bug: 1228049
QA Whiteboard: [good first verify]
Has STR: --- → yes
Reproduced the bug in firefox nightly 45.0a1 (2015-11-25) with windows 10 (64 bit)

Verified as fixed with latest firefox aurora 45.0a2 (Build ID: 20160112004003)

Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:45.0) Gecko/20100101 Firefox/45.0
QA Whiteboard: [good first verify] → [good first verify][bugday-20160113]
Status: RESOLVED → VERIFIED
Duplicate of this bug: 1227590
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.