[Webkit-unassigned] [Bug 72096] New: Web Inspector: DevTools InspectorStubs.js incorrectly converting responses to arguments

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=72096

           Summary: Web Inspector: DevTools InspectorStubs.js incorrectly
                    converting responses to arguments
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: Web Inspector
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: mikelikespie at gmail.com
                CC: timothy at apple.com, rik at webkit.org, keishi at webkit.org,
                    pmuellr at yahoo.com, joepeck at webkit.org,
                    pfeldman at chromium.org, yurys at chromium.org,
                    bweinstein at apple.com, apavlov at chromium.org,
                    loislo at chromium.org


Created an attachment (id=114638)
 --> (https://bugs.webkit.org/attachment.cgi?id=114638&action=review)
Patch to fix this bug

I've been working with the WebSocket protocol and noticed that sometimes getBodyContents is broken.

I dug into this a little more and noticed that in WebInspectorStubs.js it was just taking the result argument (object) and converting them into an array.  Well, objects aren't ordered.  The reason this worked most of the times is that most commands have 1 or 0 result arguments.

I patched CodeGeneratorInspector.py to generate correct code.

Patch is attached.


(also posted on chromium http://code.google.com/p/chromium/issues/detail?id=103881)

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.