[V8] We can merge DOMDataStore, ScopedDOMDataStore, and StaticDOMDataStore into one class
Created attachment 170531 [details] Work in progress
This currently crashes every worker test, but I'm hoping to improve that soon.
Created attachment 170552 [details] Patch
Comment on attachment 170552 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=170552&action=review LGTM. > Source/WebCore/ChangeLog:11 > + unifies all these classes into DOMDataStore itself. This makese this makese! > Source/WebCore/bindings/v8/DOMDataStore.cpp:57 > + m_domObjectMap = adoptPtr(new DOMWrapperHashMap<void>); > + m_activeDomObjectMap = adoptPtr(new DOMWrapperHashMap<void>); These are always the same for all 3, no? It seems this whole "Switch" is clearer as an if. But presumably you wanted the "forgot to add handling of the new enum value" help from the compiler?
(In reply to comment #4) > (From update of attachment 170552 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=170552&action=review > > > Source/WebCore/bindings/v8/DOMDataStore.cpp:57 > > + m_domObjectMap = adoptPtr(new DOMWrapperHashMap<void>); > > + m_activeDomObjectMap = adoptPtr(new DOMWrapperHashMap<void>); > > These are always the same for all 3, no? It seems this whole "Switch" is clearer as an if. But presumably you wanted the "forgot to add handling of the new enum value" help from the compiler? Yeah, I thought they were going to be more different. I'll change it to an if.
Created attachment 170563 [details] Patch for landing
Comment on attachment 170563 [details] Patch for landing Clearing flags on attachment: 170563 Committed r132454: <http://trac.webkit.org/changeset/132454>
All reviewed patches have been landed. Closing bug.