[webkit-reviews] review canceled: [Bug 30079] unselectable resources in resource panel : [Attachment 40791] proposed patch 2009/10/07 - a

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Oct 8 13:40:02 PDT 2009


Patrick Mueller <pmuellr at yahoo.com> has canceled Patrick Mueller
<pmuellr at yahoo.com>'s request for review:
Bug 30079: unselectable resources in resource panel
https://bugs.webkit.org/show_bug.cgi?id=30079

Attachment 40791: proposed patch 2009/10/07 - a
https://bugs.webkit.org/attachment.cgi?id=40791&action=review

------- Additional Comments from Patrick Mueller <pmuellr at yahoo.com>
The ResourceSidebarTreeElements were being created by creating a new <a>
element, and moving all the children in the existing tree element into the new
<a> element, then making the <a> element the only child of the tree element. 
Basically, wrapping the existing tree element content in an <a> element.

The <a> element isn't needed, because selection of the tree element is already
handled by the tree element itself.  The <a> element click handling was being
processed by WebInspector.documentClick(), which was then selecting a
potentially incorrect resource, if multiple resources with the same URL exist
in the resource tree.

The "invisible" CSS class is no longer used by any code in Web Inspector, as
near as I can tell (search through inspector code).

The reason the <a> element was added was to allow the resource elements to
be dragged elsewhere and rendered on the drop target as the URL to resource.
Removing the <a> element nullifies that ability.  Code was added to use HTML5
DnD to perform the same function.

Test case was added with three resource retrievals of the same URL, but
different sorts of invocation - two POSTs with different content, and a GET. 
Without the fix, only one of the three resources was selectable in the UI.


More information about the webkit-reviews mailing list