Changeset 112207 in webkit


Ignore:
Timestamp:
Mar 26, 2012 10:32:16 PM (12 years ago)
Author:
commit-queue@webkit.org
Message:

Unreviewed, rolling out r112199.
http://trac.webkit.org/changeset/112199
https://bugs.webkit.org/show_bug.cgi?id=82295

Breaks Chromium Win compilation (Requested by pfeldman on
#webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-03-26

  • bindings/scripts/CodeGeneratorV8.pm:

(GenerateConstructorCallback):
(GenerateNamedConstructorCallback):
(GenerateToV8Converters):
(GetDomMapFunction):

  • bindings/scripts/test/V8/V8Float64Array.cpp:

(WebCore::V8Float64Array::wrapSlow):

  • bindings/scripts/test/V8/V8TestActiveDOMObject.cpp:

(WebCore::V8TestActiveDOMObject::wrapSlow):

  • bindings/scripts/test/V8/V8TestCustomNamedGetter.cpp:

(WebCore::V8TestCustomNamedGetter::wrapSlow):

  • bindings/scripts/test/V8/V8TestEventConstructor.cpp:

(WebCore::V8TestEventConstructor::wrapSlow):

  • bindings/scripts/test/V8/V8TestEventTarget.cpp:

(WebCore::V8TestEventTarget::wrapSlow):

  • bindings/scripts/test/V8/V8TestInterface.cpp:

(WebCore::V8TestInterface::wrapSlow):

  • bindings/scripts/test/V8/V8TestMediaQueryListListener.cpp:

(WebCore::V8TestMediaQueryListListener::wrapSlow):

  • bindings/scripts/test/V8/V8TestNamedConstructor.cpp:

(WebCore::V8TestNamedConstructor::wrapSlow):

  • bindings/scripts/test/V8/V8TestObj.cpp:

(WebCore::V8TestObj::wrapSlow):

  • bindings/scripts/test/V8/V8TestSerializedScriptValueInterface.cpp:

(WebCore::V8TestSerializedScriptValueInterface::wrapSlow):

  • bindings/v8/V8DOMWrapper.cpp:

(WebCore::V8DOMWrapper::setJSWrapperForDOMNode):
(WebCore):

  • bindings/v8/V8DOMWrapper.h:

(WebCore):
(V8DOMWrapper):
(WebCore::V8DOMWrapper::setJSWrapperForDOMObject):
(WebCore::V8DOMWrapper::setJSWrapperForActiveDOMObject):

Location:
trunk/Source/WebCore
Files:
14 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r112203 r112207  
     12012-03-26  Sheriff Bot  <webkit.review.bot@gmail.com>
     2
     3        Unreviewed, rolling out r112199.
     4        http://trac.webkit.org/changeset/112199
     5        https://bugs.webkit.org/show_bug.cgi?id=82295
     6
     7        Breaks Chromium Win compilation (Requested by pfeldman on
     8        #webkit).
     9
     10        * bindings/scripts/CodeGeneratorV8.pm:
     11        (GenerateConstructorCallback):
     12        (GenerateNamedConstructorCallback):
     13        (GenerateToV8Converters):
     14        (GetDomMapFunction):
     15        * bindings/scripts/test/V8/V8Float64Array.cpp:
     16        (WebCore::V8Float64Array::wrapSlow):
     17        * bindings/scripts/test/V8/V8TestActiveDOMObject.cpp:
     18        (WebCore::V8TestActiveDOMObject::wrapSlow):
     19        * bindings/scripts/test/V8/V8TestCustomNamedGetter.cpp:
     20        (WebCore::V8TestCustomNamedGetter::wrapSlow):
     21        * bindings/scripts/test/V8/V8TestEventConstructor.cpp:
     22        (WebCore::V8TestEventConstructor::wrapSlow):
     23        * bindings/scripts/test/V8/V8TestEventTarget.cpp:
     24        (WebCore::V8TestEventTarget::wrapSlow):
     25        * bindings/scripts/test/V8/V8TestInterface.cpp:
     26        (WebCore::V8TestInterface::wrapSlow):
     27        * bindings/scripts/test/V8/V8TestMediaQueryListListener.cpp:
     28        (WebCore::V8TestMediaQueryListListener::wrapSlow):
     29        * bindings/scripts/test/V8/V8TestNamedConstructor.cpp:
     30        (WebCore::V8TestNamedConstructor::wrapSlow):
     31        * bindings/scripts/test/V8/V8TestObj.cpp:
     32        (WebCore::V8TestObj::wrapSlow):
     33        * bindings/scripts/test/V8/V8TestSerializedScriptValueInterface.cpp:
     34        (WebCore::V8TestSerializedScriptValueInterface::wrapSlow):
     35        * bindings/v8/V8DOMWrapper.cpp:
     36        (WebCore::V8DOMWrapper::setJSWrapperForDOMNode):
     37        (WebCore):
     38        * bindings/v8/V8DOMWrapper.h:
     39        (WebCore):
     40        (V8DOMWrapper):
     41        (WebCore::V8DOMWrapper::setJSWrapperForDOMObject):
     42        (WebCore::V8DOMWrapper::setJSWrapperForActiveDOMObject):
     43
    1442012-03-26  David Kilzer  <ddkilzer@apple.com>
    245
  • trunk/Source/WebCore/bindings/scripts/CodeGeneratorV8.pm

    r112199 r112207  
    17531753    }
    17541754
    1755     my $DOMObject = GetDomWrapperMapName($dataNode, $implClassName);
     1755    my $DOMObject = "DOMObject";
     1756    if (IsNodeSubType($dataNode)) {
     1757        $DOMObject = "DOMNode";
     1758    } elsif ($dataNode->extendedAttributes->{"ActiveDOMObject"}) {
     1759        $DOMObject = "ActiveDOMObject";
     1760    }
     1761
    17561762    push(@implContent, <<END);
    17571763
     
    19271933    }
    19281934
    1929     my $DOMObject = GetDomWrapperMapName($dataNode, $implClassName);
     1935    my $DOMObject = "DOMObject";
     1936    # A DOMObject that is an ActiveDOMObject and also a DOMNode should be treated as an DOMNode here.
     1937    # setJSWrapperForDOMNode() will look if node is active and choose correct map to add node to.
     1938    if (IsNodeSubType($dataNode)) {
     1939        $DOMObject = "DOMNode";
     1940    } elsif ($dataNode->extendedAttributes->{"ActiveDOMObject"}) {
     1941        $DOMObject = "ActiveDOMObject";
     1942    }
    19301943    push(@implContent, <<END);
    19311944
     
    30813094    my $nativeType = shift;
    30823095
    3083     my $domMapName = GetDomWrapperMapName($dataNode, $interfaceName);
     3096    my $domMapFunction = GetDomMapFunction($dataNode, $interfaceName);
    30843097    my $forceNewObjectInput = IsDOMNodeType($interfaceName) ? ", bool forceNewObject" : "";
    30853098    my $forceNewObjectCall = IsDOMNodeType($interfaceName) ? ", forceNewObject" : "";
     
    31683181    }
    31693182    push(@implContent, <<END);
    3170     V8DOMWrapper::setJSWrapperFor${domMapName}(impl, wrapperHandle);
     3183    ${domMapFunction}.set(impl.leakRef(), wrapperHandle);
    31713184    return wrapper;
    31723185}
     
    31753188
    31763189sub GetDomMapFunction
    3177 {
    3178     my $mapName = GetDomWrapperMapName(@_);
    3179     return "get${mapName}Map()";
    3180 }
    3181 
    3182 sub GetDomWrapperMapName
    31833190{
    31843191    my $dataNode = shift;
    31853192    my $type = shift;
    3186     return "DOMSVGElementInstance" if $type eq "SVGElementInstance";
    3187     return "ActiveDOMNode" if (IsNodeSubType($dataNode) && $dataNode->extendedAttributes->{"ActiveDOMObject"});
    3188     return "DOMNode" if (IsNodeSubType($dataNode));
    3189     return "ActiveDOMObject" if $dataNode->extendedAttributes->{"ActiveDOMObject"};
    3190     return "DOMObject";
     3193    return "getDOMSVGElementInstanceMap()" if $type eq "SVGElementInstance";
     3194    return "getActiveDOMNodeMap()" if (IsNodeSubType($dataNode) && $dataNode->extendedAttributes->{"ActiveDOMObject"});
     3195    return "getDOMNodeMap()" if (IsNodeSubType($dataNode));
     3196    return "getActiveDOMObjectMap()" if $dataNode->extendedAttributes->{"ActiveDOMObject"};
     3197    return "getDOMObjectMap()";
    31913198}
    31923199
  • trunk/Source/WebCore/bindings/scripts/test/V8/V8Float64Array.cpp

    r112199 r112207  
    133133    if (!hasDependentLifetime)
    134134        wrapperHandle.MarkIndependent();
    135     V8DOMWrapper::setJSWrapperForDOMObject(impl, wrapperHandle);
     135    getDOMObjectMap().set(impl.leakRef(), wrapperHandle);
    136136    return wrapper;
    137137}
  • trunk/Source/WebCore/bindings/scripts/test/V8/V8TestActiveDOMObject.cpp

    r112199 r112207  
    187187    if (!hasDependentLifetime)
    188188        wrapperHandle.MarkIndependent();
    189     V8DOMWrapper::setJSWrapperForDOMObject(impl, wrapperHandle);
     189    getDOMObjectMap().set(impl.leakRef(), wrapperHandle);
    190190    return wrapper;
    191191}
  • trunk/Source/WebCore/bindings/scripts/test/V8/V8TestCustomNamedGetter.cpp

    r112199 r112207  
    123123    if (!hasDependentLifetime)
    124124        wrapperHandle.MarkIndependent();
    125     V8DOMWrapper::setJSWrapperForDOMObject(impl, wrapperHandle);
     125    getDOMObjectMap().set(impl.leakRef(), wrapperHandle);
    126126    return wrapper;
    127127}
  • trunk/Source/WebCore/bindings/scripts/test/V8/V8TestEventConstructor.cpp

    r112199 r112207  
    158158    if (!hasDependentLifetime)
    159159        wrapperHandle.MarkIndependent();
    160     V8DOMWrapper::setJSWrapperForDOMObject(impl, wrapperHandle);
     160    getDOMObjectMap().set(impl.leakRef(), wrapperHandle);
    161161    return wrapper;
    162162}
  • trunk/Source/WebCore/bindings/scripts/test/V8/V8TestEventTarget.cpp

    r112199 r112207  
    186186    if (!hasDependentLifetime)
    187187        wrapperHandle.MarkIndependent();
    188     V8DOMWrapper::setJSWrapperForDOMObject(impl, wrapperHandle);
     188    getDOMObjectMap().set(impl.leakRef(), wrapperHandle);
    189189    return wrapper;
    190190}
  • trunk/Source/WebCore/bindings/scripts/test/V8/V8TestInterface.cpp

    r112199 r112207  
    318318    if (!hasDependentLifetime)
    319319        wrapperHandle.MarkIndependent();
    320     V8DOMWrapper::setJSWrapperForActiveDOMObject(impl, wrapperHandle);
     320    getActiveDOMObjectMap().set(impl.leakRef(), wrapperHandle);
    321321    return wrapper;
    322322}
  • trunk/Source/WebCore/bindings/scripts/test/V8/V8TestMediaQueryListListener.cpp

    r112199 r112207  
    123123    if (!hasDependentLifetime)
    124124        wrapperHandle.MarkIndependent();
    125     V8DOMWrapper::setJSWrapperForDOMObject(impl, wrapperHandle);
     125    getDOMObjectMap().set(impl.leakRef(), wrapperHandle);
    126126    return wrapper;
    127127}
  • trunk/Source/WebCore/bindings/scripts/test/V8/V8TestNamedConstructor.cpp

    r112199 r112207  
    167167    if (!hasDependentLifetime)
    168168        wrapperHandle.MarkIndependent();
    169     V8DOMWrapper::setJSWrapperForActiveDOMObject(impl, wrapperHandle);
     169    getActiveDOMObjectMap().set(impl.leakRef(), wrapperHandle);
    170170    return wrapper;
    171171}
  • trunk/Source/WebCore/bindings/scripts/test/V8/V8TestObj.cpp

    r112199 r112207  
    21732173    if (!hasDependentLifetime)
    21742174        wrapperHandle.MarkIndependent();
    2175     V8DOMWrapper::setJSWrapperForDOMObject(impl, wrapperHandle);
     2175    getDOMObjectMap().set(impl.leakRef(), wrapperHandle);
    21762176    return wrapper;
    21772177}
  • trunk/Source/WebCore/bindings/scripts/test/V8/V8TestSerializedScriptValueInterface.cpp

    r112199 r112207  
    289289    if (!hasDependentLifetime)
    290290        wrapperHandle.MarkIndependent();
    291     V8DOMWrapper::setJSWrapperForDOMObject(impl, wrapperHandle);
     291    getDOMObjectMap().set(impl.leakRef(), wrapperHandle);
    292292    return wrapper;
    293293}
  • trunk/Source/WebCore/bindings/v8/V8DOMWrapper.cpp

    r112199 r112207  
    6767namespace WebCore {
    6868
     69void V8DOMWrapper::setJSWrapperForDOMNode(PassRefPtr<Node> node, v8::Persistent<v8::Object> wrapper)
     70{
     71    ASSERT(maybeDOMWrapper(wrapper));
     72    if (node->isActiveNode())
     73        getActiveDOMNodeMap().set(node.leakRef(), wrapper);
     74    else
     75        getDOMNodeMap().set(node.leakRef(), wrapper);
     76}
     77
    6978v8::Local<v8::Function> V8DOMWrapper::getConstructor(WrapperTypeInfo* type, v8::Handle<v8::Value> objectPrototype)
    7079{
  • trunk/Source/WebCore/bindings/v8/V8DOMWrapper.h

    r112199 r112207  
    5555    class EventTarget;
    5656    class Frame;
    57     class SVGElementInstance;
     57    class Node;
    5858    class V8Proxy;
    5959    class WorkerContext;
     
    111111        template<typename T> static void setJSWrapperForActiveDOMObject(PassRefPtr<T>, v8::Persistent<v8::Object>);
    112112        static void setJSWrapperForDOMNode(PassRefPtr<Node>, v8::Persistent<v8::Object>);
    113         static void setJSWrapperForActiveDOMNode(PassRefPtr<Node>, v8::Persistent<v8::Object>);
    114 #if ENABLE(SVG)
    115         static void setJSWrapperForDOMSVGElementInstance(PassRefPtr<SVGElementInstance>, v8::Persistent<v8::Object>);
    116 #endif
    117113
    118114        static bool isValidDOMObject(v8::Handle<v8::Value>);
     
    155151
    156152    template<typename T>
    157     inline void V8DOMWrapper::setJSWrapperForDOMObject(PassRefPtr<T> object, v8::Persistent<v8::Object> wrapper)
     153    void V8DOMWrapper::setJSWrapperForDOMObject(PassRefPtr<T> object, v8::Persistent<v8::Object> wrapper)
    158154    {
    159155        ASSERT(maybeDOMWrapper(wrapper));
     
    163159
    164160    template<typename T>
    165     inline void V8DOMWrapper::setJSWrapperForActiveDOMObject(PassRefPtr<T> object, v8::Persistent<v8::Object> wrapper)
     161    void V8DOMWrapper::setJSWrapperForActiveDOMObject(PassRefPtr<T> object, v8::Persistent<v8::Object> wrapper)
    166162    {
    167163        ASSERT(maybeDOMWrapper(wrapper));
     
    169165        getActiveDOMObjectMap().set(object.leakRef(), wrapper);
    170166    }
    171 
    172     inline void V8DOMWrapper::setJSWrapperForDOMNode(PassRefPtr<Node> node, v8::Persistent<v8::Object> wrapper)
    173     {
    174         ASSERT(maybeDOMWrapper(wrapper));
    175         ASSERT(!domWrapperType(wrapper)->toActiveDOMObjectFunction);
    176         ASSERT(!node->isActiveNode());
    177         getDOMNodeMap().set(node.leakRef(), wrapper);
    178     }
    179 
    180     inline void V8DOMWrapper::setJSWrapperForActiveDOMNode(PassRefPtr<Node> node, v8::Persistent<v8::Object> wrapper)
    181     {
    182         ASSERT(maybeDOMWrapper(wrapper));
    183         ASSERT(domWrapperType(wrapper)->toActiveDOMObjectFunction);
    184         ASSERT(node->isActiveNode());
    185         getActiveDOMNodeMap().set(node.leakRef(), wrapper);
    186     }
    187 
    188 #if ENABLE(SVG)
    189     inline void V8DOMWrapper::setJSWrapperForDOMSVGElementInstance(PassRefPtr<SVGElementInstance> element, v8::Persistent<v8::Object> wrapper)
    190     {
    191         ASSERT(maybeDOMWrapper(wrapper));
    192         getDOMSVGElementInstanceMap().set(element.leakRef(), wrapper);
    193     }
    194 #endif
    195 
    196167} // namespace WebCore
    197168
Note: See TracChangeset for help on using the changeset viewer.