[V8] DOMData is no longer needed
Created attachment 170524 [details] Patch
Comment on attachment 170524 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=170524&action=review LGTM. > Source/WebCore/bindings/v8/V8DOMMap.cpp:91 > + DOMDataStore* store = DOMDataStore::from(v8::Isolate::GetCurrent()); This pattern is confusing to me. It's unclear who owns the object. Perhaps sharedInstanceFor? or lookupForInstance? or even just changing it back to a reference makes it clear that the caller isn't responsible for the memory (or handling null!)
Created attachment 170548 [details] Patch for landing
Comment on attachment 170548 [details] Patch for landing Clearing flags on attachment: 170548 Committed r132447: <http://trac.webkit.org/changeset/132447>
All reviewed patches have been landed. Closing bug.