1187482 - Inspector stops showing children of an element if highlighter targets xul:thumb inside its overflow scrollbar

Status: VERIFIED FIXED

(DevTools :: Inspector, defect)

Description

[Boris Zbarsky [:bzbarsky]]

STEPS TO REPRODUCE:

1)  Load https://bug1187172.bmoattachments.org/attachment.cgi?id=8638275 
2)  Open devtools
3)  Select the "Inspector" tool.
4)  Click the triangle next to the <div class="row"> to expand it.
5)  Click the "pick an element" button.
6)  Mouse over the first cat picture.

EXPECTED RESULTS: Can click that picture to get something useful to happen.

ACTUAL RESULTS: The kids of the <div class="row"> disappear in the inspector view.

Brian, please let me know whether you still need a video showing this happening now that I've added the step 4 in there which you may not have been doing...

Comment 1

[Brian Grinstead [:bgrins]]

Attachment: cat-testcase.mov

Maybe I'm missing a step, but this screencast is on a nightly build on OSX - 42.0a1 (2015-07-27) and I'm not seeing the issue.  Tried also in a debug build since I figured you and tromey might have been testing with that, but got the same results.

Comment 2

[Brian Grinstead [:bgrins]]

Chatted with Tom, and it sounds like the STR in the video look right but I'm seeing different results than in his build.  Can you reproduce this in a clean profile?

Comment 3

[Boris Zbarsky [:bzbarsky]]

Attachment: Screen recording showing the problem

I was definitely testing a regular nightly.

I just checked again in a Mac 2015-07-27 nightly and I'm totally seeing the problem.  This is the screen recording.  I made sure to test with a clean profile, fwiw.

Comment 4

[Tom Tromey :tromey]

    (In reply to Brian Grinstead [:bgrins] from comment #2)
    > Chatted with Tom, and it sounds like the STR in the video look right but I'm
    > seeing different results than in his build.  Can you reproduce this in a
    > clean profile?

I built today's fx-team and was able to reproduce it with a clean profile.

For me it seems to depend on the path the mouse takes to the cat picture.
If I move the mouse outside the window, then up vertically, then over the
cat, I don't see the bug.  But if I move the mouse directly up from the
inspector, over other content, then I can reproduce.

After experimenting some more the problem seems to arise when the mouse
moves over the horizontal scrollbar beneath the cats.  According to a tooltip
this is a "xul:slider", but it doesn't seem to appear in the document.

Comment 5

[Brian Grinstead [:bgrins]]

(In reply to Tom Tromey :tromey from comment #4)
> After experimenting some more the problem seems to arise when the mouse
> moves over the horizontal scrollbar beneath the cats.  According to a tooltip
> this is a "xul:slider", but it doesn't seem to appear in the document.

I see, I can reproduce now by scrolling horizontally while inspecting, and then moving the cursor over the visible scrollbar.  It does indeed seem that the highlighter is inspecting some anonymous content that it shouldn't be (xul:thumb).

Comment 6

[Brian Grinstead [:bgrins]]

Easier STR:

1) Open this page

data:text/html,<div style="overflow:scroll;height:50px;width:50px;border:solid 1px;"><div style="height: 100px; width: 100px"></div></div>

2) Press ctrl+shift+c / cmd+shift+c to start inspecting mode
3) Hover over the div and scroll the overflow region around while still inspecting
4) Move the mouse on top of the scrollbar thumb

The child element of the parent div disappears

Comment 7

[Brian Grinstead [:bgrins]]

This is similar to Bug 1093875.  Basically, the walker isn't filtering out the XBL anonymous nodes.

Unfortunately filtering out XBL content from the walker breaks inspection on the yellow bar on a page like this: http://limpet.net/mbrubeck/atom.xml.  In that case the page is somehow embedding XBL content.  So we need to be able to distinguish this case from that one.

Comment 8

[Brian Grinstead [:bgrins]]

I'm fairly sure this is a regression from Bug 1168689, I can't reproduce it in Beta 40 but can in Dev Ed 41

Comment 9

[Brian Grinstead [:bgrins]]

Looks like the feed subscribe page is special cased: https://dxr.mozilla.org/mozilla-central/source/dom/xul/nsXULElement.cpp#796.

According to https://dxr.mozilla.org/mozilla-central/source/dom/xul/nsXULElement.cpp?from=isinfeedsubscribeline#873-876 the only places that XUL elements are pulled into content is IsInVideoControls or IsInFeedSubscribeLine.  These could probably be special cased in our Walker as well and then simplify our logic for skipping nodes so we can go back to skipping all anonymous elements in a non-XUL document.

Comment 10

[Brian Grinstead [:bgrins]]

Another simple test case: data:text/html,<select multiple><option>1</option><option>1</option><option>1</option><option>1</option><option>1</option><option>1</option><option>1</option><option>1</option></select>

Comment 11

[Brian Grinstead [:bgrins]]

Attachment: MozReview Request: Bug 1187482 - Skip XBL and native anonymous content inside non-XUL documents;r=jryans

Bug 1187482 - Skip XBL and native anonymous content inside non-XUL documents;r=jryans

This will allow the walker to skip scrollbar elements passed in from the highlighter.
We used to allow anonymous content just so that feedSubscribeLine contents could be
inspected, but that's much less important than the problem it's causing.  With this
change we still show ::before/::after but nothing else.

Note that this doesn't affect the Browser Toolbox at all, which uses another filter.

Comment 12

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Comment on attachment 8641689 [details]
MozReview Request: Bug 1187482 - Skip XBL and native anonymous content inside non-XUL documents;r=jryans

https://reviewboard.mozilla.org/r/14591/#review13213

Seems to work well in my testing.

::: toolkit/devtools/server/actors/inspector.js:3891
(Diff revision 1)
> -  if (LayoutHelpers.isNativeAnonymous(aNode) &&
> +    // controls, scrollbars, video controls, etc (see bug 11874820).

Nit: Incorrect bug number

Comment 13

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/b06e4e859f5f

Comment 14

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/b06e4e859f5f

Comment 15

[Nazir Ahmed Sabbir [:NaSb]]

Reproduced this bug in Nightly 42.0a1 (2015-07-24) (Build ID: 20150724030210) on Linux,64 bit with the comment 0's instruction!

This Bug is now verified as fixed on Latest Firefox Beta 42.0b7

Build ID 	20151015151621
User Agent 	Mozilla/5.0 (X11; Linux x86_64; rv:42.0) Gecko/20100101 Firefox/42.0

Comment 16

[Petruta Horea [:phorea], Desktop Test Engineering]

Thanks, Nazir!
Marking as verified.