Closed Bug 1260043 Opened 4 years ago Closed 4 years ago

"Toggle rulers for the page" is broken if toolbox is detached


(DevTools Graveyard :: Graphic Commandline and Toolbar, defect, P2)



(firefox48 affected, firefox49 fixed)

Firefox 49
Tracking Status
firefox48 --- affected
firefox49 --- fixed


(Reporter: arni2033, Unassigned)




(1 file)

>>>   My Info:   Win7_64, Nightly 48, 32bit, ID 20160326030430
1. Open in a new tab
2. Open devtools, dock it to the bottom side of the window 
3. Open in a new tab   (optional: open devtools in that tab)
4. Switch to the tab from Step 1, detach toolbox in separate window
5. Switch to the tab from Step 3
6. Switch to devtools window from Step 4
7. Click "Toggle rulers for the page" button
8. Switch to the tab from Step 1

 No rulers

ER:  Either X or Y or Z
 X) After Step 8 rulers should be toggled for the tab from Step 1
 Y) After Step 7 browser window should be placed below toolbox window and the tab from Step 1
    should be selected
 Z) After Step 7 an alert/popup panel/something else should say that tab from Step 1 is not visible

See also explanation of (Y) in bug 1251863 comment 6
See Also: → 1260044
Component: Developer Tools → Developer Tools: Graphic Commandline and Toolbar
See Also: → 1259391
Flags: needinfo?(zer0)
See Also: → 1267219
Depends on: 1267219
See Also: 1267219
I filed bug 1267219, fixing that should fixes both this bug and bug 1260044.
Flags: needinfo?(zer0)
Comment on attachment 8758703 [details]
MozReview Request: Bug 1260043 - removed `doc.hidden` check from markup/utils; r=pbro

I think Alex would be interested in reviewing this since he was the one who worked on this part of the code if I remember correctly.
Alex, you should read Bug 1267219 comment 11 before I think.
Attachment #8758703 - Flags: review?(pbrosset) → review?(poirot.alex)
Attachment #8758703 - Flags: review?(poirot.alex) → review+
Comment on attachment 8758703 [details]
MozReview Request: Bug 1260043 - removed `doc.hidden` check from markup/utils; r=pbro

I can't find any safe check that isn't document.hidden.
Nor can I find of any real case where similar thing would happen on desktop.
But that's great to keep this comment in case it starts to throw again.

The only case where it may fail is for documents from the bfcache, but I don't see why we would have an highlighter against such document as TabActor throws events when we switch into/off the bfcache...

So if try is green, let's just remove this.
The orange doesn't seems related to this patch:

So I'm going to land it. Thanks for the review Alex!
Duplicate of this bug: 1260044
Pushed by
removed `doc.hidden` check from markup/utils; r=ochameau
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Product: Firefox → DevTools
Product: DevTools → DevTools Graveyard
You need to log in before you can comment on or make changes to this bug.