Changeset 142128 in webkit


Ignore:
Timestamp:
Feb 7, 2013 8:30:20 AM (11 years ago)
Author:
vsevik@chromium.org
Message:

Web Inspector: [Regression] Map.size() returns negative values.
​https://bugs.webkit.org/show_bug.cgi?id=109174

Reviewed by Yury Semikhatsky.

Source/WebCore:

  • inspector/front-end/utilities.js:

LayoutTests:

  • inspector/map-expected.txt:
  • inspector/map.html:
Location:
trunk
Files:
5 edited

Legend:

Unmodified
Added
Removed

  • trunk/LayoutTests/ChangeLog

    r142122 r142128  
     12013-02-07  Vsevolod Vlasov  <vsevik@chromium.org>
     2
     3        Web Inspector: [Regression] Map.size() returns negative values.
     4        https://bugs.webkit.org/show_bug.cgi?id=109174
     5
     6        Reviewed by Yury Semikhatsky.
     7
     8        * inspector/map-expected.txt:
     9        * inspector/map.html:
     10
    1112013-02-07  Kent Tamura  <tkent@chromium.org>
    212

  • trunk/LayoutTests/inspector/map-expected.txt

    r102447 r142128  
    66    key1 : undefined
    77    key2 : undefined
     8    size : 0
    89  Second map:
    910    key1 : undefined
    1011    key2 : undefined
     12    size : 0
    1113
    1214Dumping maps:
    … …  
    1416    key1 : 2
    1517    key2 : undefined
     18    size : 1
    1619  Second map:
    1720    key1 : undefined
    1821    key2 : undefined
     22    size : 0
    1923
    2024Dumping maps:
    … …  
    2226    key1 : 1
    2327    key2 : undefined
     28    size : 1
    2429  Second map:
    2530    key1 : undefined
    2631    key2 : undefined
     32    size : 0
    2733
    2834Dumping maps:
    … …  
    3036    key1 : 1
    3137    key2 : undefined
     38    size : 1
    3239  Second map:
    3340    key1 : 1
    3441    key2 : undefined
     42    size : 1
    3543
    3644Dumping maps:
    … …  
    3846    key1 : 1
    3947    key2 : undefined
     48    size : 1
    4049  Second map:
    4150    key1 : 2
    4251    key2 : undefined
     52    size : 1
    4353
    4454Dumping maps:
    … …  
    4656    key1 : 1
    4757    key2 : 2
     58    size : 2
    4859  Second map:
    4960    key1 : 2
    5061    key2 : undefined
     62    size : 1
    5163
    5264Dumping maps:
    … …  
    5466    key1 : 1
    5567    key2 : 2
     68    size : 2
    5669  Second map:
    5770    key1 : 2
    5871    key2 : 1
     72    size : 2
    5973
    6074Dumping maps:
    … …  
    6276    key1 : 1
    6377    key2 : undefined
     78    size : 1
    6479  Second map:
    6580    key1 : 2
    6681    key2 : 1
     82    size : 2
    6783
    6884Dumping maps:
    … …  
    7086    key1 : 1
    7187    key2 : undefined
     88    size : 1
    7289  Second map:
    7390    key1 : 2
    7491    key2 : undefined
     92    size : 1
    7593
    7694Dumping maps:
    … …  
    7896    key1 : 1
    7997    key2 : undefined
     98    size : 1
    8099  Second map:
    81100    key1 : 2
    82101    key2 : undefined
     102    size : 1
    83103
    84104Dumping maps:
    … …  
    86106    key1 : undefined
    87107    key2 : undefined
     108    size : 0
    88109  Second map:
    89110    key1 : 2
    90111    key2 : undefined
     112    size : 1
    91113
    92114Dumping maps:
    … …  
    94116    key1 : undefined
    95117    key2 : undefined
     118    size : 0
    96119  Second map:
    97120    key1 : undefined
    98121    key2 : undefined
     122    size : 0
    99123
    100124

  • trunk/LayoutTests/inspector/map.html

    r102447 r142128  
    1212        InspectorTest.addResult("    key1 : " + map1.get(key1));
    1313        InspectorTest.addResult("    key2 : " + map1.get(key2));
     14        InspectorTest.addResult("    size : " + map1.size());
    1415        InspectorTest.addResult("  Second map:");
    1516        InspectorTest.addResult("    key1 : " + map2.get(key1));
    1617        InspectorTest.addResult("    key2 : " + map2.get(key2));
     18        InspectorTest.addResult("    size : " + map2.size());
    1719        InspectorTest.addResult("");
    1820    }

  • trunk/Source/WebCore/ChangeLog

    r142127 r142128  
     12013-02-07  Vsevolod Vlasov  <vsevik@chromium.org>
     2
     3        Web Inspector: [Regression] Map.size() returns negative values.
     4        https://bugs.webkit.org/show_bug.cgi?id=109174
     5
     6        Reviewed by Yury Semikhatsky.
     7
     8        * inspector/front-end/utilities.js:
     9
    1102013-02-07  Pavel Feldman  <pfeldman@chromium.org>
    211

  • trunk/Source/WebCore/inspector/front-end/utilities.js

    r141594 r142128  
    725725    {
    726726        var result = this._map[key.__identifier];
     727        if (!result)
     728            return undefined;
     729        --this._size;
    727730        delete this._map[key.__identifier];
    728         --this._size;
    729         return result ? result[1] : undefined;
     731        return result[1];
    730732    },
    731733
Note: See TracChangeset for help on using the changeset viewer.