Bug 140513 – Web Inspector: Reopening the inspector can selects a sub-frame instead of the Main Frame's DOM

• Component: Web Inspector
• Status: RESOLVED FIXED

[Joseph Pecoraro]

* SUMMARY
Reopening the inspector can selects a sub-frame instead of the Main Frame's DOM. Very annoying!

* STEPS TO REPRODUCE
1. Load http://www.nytimes.com/interactive/2015/01/09/sports/el-capitan-2d-image.html
2. Inspect an element on the page
3. Close Inspector
4. Re-open Inspector
  => sub-frame selected instead of main frame

* NOTES
- Somewhat related to bug 139593

[Radar WebKit Bug Importer]

<rdar://problem/19490093>

[Joseph Pecoraro]

Created attachment 244735 [details]
[PATCH] Proposed Fix

I think ideally we would have a way to uniquely identify sub-frames, but really we don't have much more information than their URL and maybe frame index (which could change). Specifically identifying the main frame I think is a good compromise, at least that is guaranteed to be unique.

[Simon Fraser (smfr)]

Comment on attachment 244735 [details]
[PATCH] Proposed Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=244735&action=review

> Source/WebInspectorUI/ChangeLog:11
> +        Specifically identify the main frame, so that any sub-frames with the same
> +        URL, don't get choosen over the main frame when restoring state from cookie.

Why do subframes get the same URL? It seems very unlikely that sites are loading the same url in subframes.

[Joseph Pecoraro]

Thats what I want to investigate in bug 139593.

[WebKit Commit Bot]

Comment on attachment 244735 [details]
[PATCH] Proposed Fix

Clearing flags on attachment: 244735

Committed r178569: <http://trac.webkit.org/changeset/178569>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.