772497 - [layout view] doesn't like SVG documents

Status: RESOLVED WORKSFORME

(DevTools :: Inspector, defect, P2)

Description

[Loic]

Attachment: Testcase (unzip 3 files)

I tested the testcase from bug 771520 (issue with SVG resizing) and I met this weird bug.

STR:
1) Download the testcase (.zip) attached to this bug and unzip the 3 files.
2) Open a 1st tab (load about:blank e.g.)
3) Open a 2nd tab and load the testcase file 'test.xhtml'
4) Inspect Element
5) Click on 'object' then 'svg' in the Inspector bar (or directly on 'svg' if it's already displayed) 
6) Switch to the 1st tab
7) Switch to the 2nd tab

1st result: Elements are not displayed anymore in the Inspector bar.

8) Click on button 'Select element with mouse'
9) Switch to the 1st tab
10) Switch to the 2nd tab
11) Repeat steps #8 to #10 a few times (3 times should be enough) to keep checked the button 'Select element with mouse'

2nd result: Switch to the 1st tab or open a 3rd tab, Inspector stays enabled and can't be closed (even from the Web Developer menu).

Comment 1

[Paul Rouget [:paul]]

Does this still happen in the very latest nightly? (with bug 769299 fixed)

Comment 2

[Loic]

Yes, each time I report a bug, I check Nightly is up-to-date. ;)
Tested with:
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:16.0) Gecko/16.0 Firefox/16.0
Built from http://hg.mozilla.org/mozilla-central/rev/02b26fb307b4

Anyway the underlying bug 771520 needs to be investigated, because Inspector changes the SVG size when the user opens it.

Comment 3

[Loic]

It looks like the bug appears each time the Inspector has the focus on SVG element embedded with <object>.

Comment 4

[Loic]

Some errors logged in the console:

Timestamp: 24/08/2012 22:30:42
Error: TypeError: aIframe.contentWindow is undefined
Source File: resource:///modules/devtools/LayoutHelpers.jsm
Line: 144

Comment 5

[Dave Camp (:dcamp)]

The patch in bug 746197 should be tested to make sure it fixes this.

Comment 6

[Paul Rouget [:paul]]

We can now close the inspector, but we still have an exception in the error console:

> Error: TypeError: aIframe.contentWindow is undefined
> Source File: resource:///modules/devtools/LayoutHelpers.jsm
> Line: 154

2 things to fix here:
- the layout view should support SVG documents
- an exception in a view in the inspector should not break the other views

Comment 7

[Luke]

Is this only with linked object tag's svg? I had also had problems inspecting svg, while it was linked from an embed tag.

Comment 8

[Patrick Brosset <:pbro>]

Works for me fine with FF48. Not sure what fixed this, but the inspector has changed a lot in the past 4 years, so I'm not surprised this works now.
Please re-open if it still occurs for you.