Changeset 51413 in webkit


Ignore:
Timestamp:
Nov 26, 2009 5:20:26 AM (14 years ago)
Author:
eric@webkit.org
Message:

2009-11-26 Anton Muhin <antonm@chromium.org>

Reviewed by Adam Barth.

Do not check if the thread is main or not when accessing DOMDataStore as currently in Chromium WebKit is used in main thread only.
https://bugs.webkit.org/show_bug.cgi?id=31877

Covered by layout tests and buildbots.

  • bindings/v8/V8DOMMap.cpp: (WebCore::getDOMNodeMap): (WebCore::getDOMObjectMap): (WebCore::getActiveDOMObjectMap): (WebCore::getDOMSVGElementInstanceMap): (WebCore::getDOMSVGObjectWithContextMap):
Location:
trunk/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/WebCore/ChangeLog

    r51412 r51413  
     12009-11-26  Anton Muhin  <antonm@chromium.org>
     2
     3        Reviewed by Adam Barth.
     4
     5        Do not check if the thread is main or not when accessing DOMDataStore as currently in Chromium WebKit is used in main thread only.
     6        https://bugs.webkit.org/show_bug.cgi?id=31877
     7
     8        Covered by layout tests and buildbots.
     9
     10        * bindings/v8/V8DOMMap.cpp:
     11        (WebCore::getDOMNodeMap):
     12        (WebCore::getDOMObjectMap):
     13        (WebCore::getActiveDOMObjectMap):
     14        (WebCore::getDOMSVGElementInstanceMap):
     15        (WebCore::getDOMSVGObjectWithContextMap):
     16
    1172009-11-26  Girish Ramakrishnan  <girish@forwardbias.in>
    218
  • trunk/WebCore/bindings/v8/V8DOMMap.cpp

    r50826 r51413  
    4949}
    5050
     51static inline DOMDataStore& getDOMDataStore()
     52{
     53    ASSERT(WTF::isMainThread());  // As of now, we must be always on the main thread.
     54    return MainThreadDOMData::getCurrentMainThreadStore();
     55}
     56
    5157DOMWrapperMap<Node>& getDOMNodeMap()
    5258{
    53     return MainThreadDOMData::getCurrentMainThreadStore().domNodeMap();
     59    return getDOMDataStore().domNodeMap();
    5460}
    5561
    5662DOMWrapperMap<void>& getDOMObjectMap()
    5763{
    58     return DOMData::getCurrent()->getStore().domObjectMap();
     64    return getDOMDataStore().domObjectMap();
    5965}
    6066
    6167DOMWrapperMap<void>& getActiveDOMObjectMap()
    6268{
    63     return DOMData::getCurrent()->getStore().activeDomObjectMap();
     69    return getDOMDataStore().activeDomObjectMap();
    6470}
    6571
     
    6874DOMWrapperMap<SVGElementInstance>& getDOMSVGElementInstanceMap()
    6975{
    70     return DOMData::getCurrent()->getStore().domSvgElementInstanceMap();
     76    return getDOMDataStore().domSvgElementInstanceMap();
    7177}
    7278
     
    7480DOMWrapperMap<void>& getDOMSVGObjectWithContextMap()
    7581{
    76     return DOMData::getCurrent()->getStore().domSvgObjectWithContextMap();
     82    return getDOMDataStore().domSvgObjectWithContextMap();
    7783}
    7884
Note: See TracChangeset for help on using the changeset viewer.