[Webkit-unassigned] [Bug 31877] [v8] Do not check the thread when accessing DOMDataStore

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=31877





--- Comment #6 from anton muhin <antonm at chromium.org>  2009-11-25 11:42:31 PST ---
(In reply to comment #3)
> (From update of attachment 43854 [details])
> > Index: WebCore/bindings/v8/V8DOMMap.cpp
> ...
> > +namespace {
> > +
> > +static inline DOMDataStore& getDOMDataStore()
> > +{
> > +    ASSERT(WTF::isMainThread());  // As of now, we must be always on the main thread.
> > +    return MainThreadDOMData::getCurrentMainThreadStore();
> > +}
> > +
> > +}
> 
> nit: The anonymous namespace is redundant.  A static function already has
> internal linkage.  I would just delete the anonymous namespace.

Oh, sure, sorry, those styles got mixed in my head.

Thanks a lot for a comments/review, Darin and Adam.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.