Changeset 85673 in webkit


Ignore:
Timestamp:
May 3, 2011 3:22:49 PM (13 years ago)
Author:
antonm@chromium.org
Message:

2011-05-03 Anton Muhin <antonm@chromium.org>

Reviewed by Yury Semikhatsky.

[v8] remove an ASSERT from grouping logic
https://bugs.webkit.org/show_bug.cgi?id=60024

This ASSERT was exploratory. Alas, right now I am aware of no easy way to
repro it. Removing for now for greener bots.

No new tests, only an ASSERT removal.

  • bindings/v8/V8GCController.cpp: (WebCore::calculateGroupId):
Location:
trunk/Source/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r85669 r85673  
     12011-05-03  Anton Muhin  <antonm@chromium.org>
     2
     3        Reviewed by Yury Semikhatsky.
     4
     5        [v8] remove an ASSERT from grouping logic
     6        https://bugs.webkit.org/show_bug.cgi?id=60024
     7
     8        This ASSERT was exploratory.  Alas, right now I am aware of no easy way to
     9        repro it.  Removing for now for greener bots.
     10
     11        No new tests, only an ASSERT removal.
     12
     13        * bindings/v8/V8GCController.cpp:
     14        (WebCore::calculateGroupId):
     15
    1162011-05-03  Ryosuke Niwa  <rniwa@webkit.org>
    217
  • trunk/Source/WebCore/bindings/v8/V8GCController.cpp

    r84749 r85673  
    297297        if (current->isMutableStyleDeclaration()) {
    298298            CSSMutableStyleDeclaration* cssMutableStyleDeclaration = static_cast<CSSMutableStyleDeclaration*>(current);
    299             if (cssMutableStyleDeclaration->isInlineStyleDeclaration()) {
    300                 ASSERT(cssMutableStyleDeclaration->parent()->isStyleSheet());
     299            if (cssMutableStyleDeclaration->isInlineStyleDeclaration())
    301300                return calculateGroupId(cssMutableStyleDeclaration->node());
    302             }
    303301            // Either we have no parent, or this parent is a CSSRule.
    304302            ASSERT(cssMutableStyleDeclaration->parent() == cssMutableStyleDeclaration->parentRule());
Note: See TracChangeset for help on using the changeset viewer.