Bug 103222

Summary: Web Inspector: object preview does not render node id, className; logs too many functions for jQuery.
Product: WebKit Reporter: Pavel Feldman <pfeldman>
Component: Web Inspector (Deprecated)Assignee: Pavel Feldman <pfeldman>
Status: RESOLVED FIXED    
Severity: Normal CC: apavlov, keishi, loislo, pfeldman, pmuellr, vsevik, web-inspector-bugs, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Patch yurys: review+, buildbot: commit-queue-

Pavel Feldman
Reported 2012-11-26 00:59:10 PST
Attachments
Patch (16.09 KB, patch)
2012-11-26 01:10 PST, Pavel Feldman
no flags
Patch (23.14 KB, patch)
2012-11-26 02:12 PST, Pavel Feldman
yurys: review+
buildbot: commit-queue-
Pavel Feldman
Comment 1 2012-11-26 01:10:38 PST
Build Bot
Comment 2 2012-11-26 01:43:44 PST
Comment on attachment 175939 [details] Patch Attachment 175939 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/15003033 New failing tests: inspector/console/command-line-api-inspect.html inspector/console/console-format-collections.html
Yury Semikhatsky
Comment 3 2012-11-26 02:01:39 PST
Comment on attachment 175939 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=175939&action=review > Source/WebCore/inspector/InspectorOverlayPage.html:91 > + /* Keep this in sync with view-source.css (.webkit-html-attribute-value) */ can we declare this style once and enumerate all selectors, like this: #node-id, .webkit-html-attribute-value, .node-id {...} ? > Source/WebCore/inspector/InspectorOverlayPage.html:349 > + document.getElementById("class-name").textContent = className || ""; className || "" -> className ?
Pavel Feldman
Comment 4 2012-11-26 02:12:20 PST
Build Bot
Comment 5 2012-11-26 03:10:50 PST
Comment on attachment 175945 [details] Patch Attachment 175945 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/14984594 New failing tests: inspector/console/console-format-collections.html
Pavel Feldman
Comment 6 2012-11-26 06:02:36 PST
Pavel Feldman
Comment 7 2012-11-26 07:19:26 PST
Note You need to log in before you can comment on or make changes to this bug.