Bug 109174

Summary: Web Inspector: [Regression] Map.size() returns negative values.
Product: WebKit Reporter: Vsevolod Vlasov <vsevik>
Component: Web Inspector (Deprecated)Assignee: Vsevolod Vlasov <vsevik>
Status: RESOLVED FIXED    
Severity: Normal CC: apavlov, keishi, loislo, pfeldman, pmuellr, vsevik, web-inspector-bugs, webkit.review.bot, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Patch none

Description Vsevolod Vlasov 2013-02-07 04:11:38 PST
Patch to follow
Comment 1 Vsevolod Vlasov 2013-02-07 04:12:53 PST
Created attachment 187056 [details]
Patch
Comment 2 Vsevolod Vlasov 2013-02-07 04:18:50 PST
Created attachment 187058 [details]
Patch
Comment 3 WebKit Review Bot 2013-02-07 08:31:57 PST
Comment on attachment 187058 [details]
Patch

Clearing flags on attachment: 187058

Committed r142128: <http://trac.webkit.org/changeset/142128>
Comment 4 WebKit Review Bot 2013-02-07 08:32:00 PST
All reviewed patches have been landed.  Closing bug.
Comment 5 Pavel Feldman 2013-02-12 13:06:37 PST
Comment on attachment 187058 [details]
Patch

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

> Source/WebCore/inspector/front-end/utilities.js:727
> +        if (!result)

Are you saying we do not allow storing "null", empty string, etc as values?
Comment 6 Pavel Feldman 2013-02-12 13:21:44 PST
Comment on attachment 187058 [details]
Patch

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

>> Source/WebCore/inspector/front-end/utilities.js:727
>> +        if (!result)
> 
> Are you saying we do not allow storing "null", empty string, etc as values?

Ignore me, now I see that you store pairs as values.