I found another place with raw memory management in RenderBlock code. patch to follow
Created attachment 176869 [details] Patch
Comment on attachment 176869 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=176869&action=review > Source/WebCore/rendering/RenderBlock.cpp:3183 > continuations = new ListHashSet<RenderInline*>; > - table->set(this, continuations); > + table->set(this, adoptPtr(continuations)); Consider merging these lines. We prefer to call adoptPtr immediately after calling new: adoptPtr(new Foo(....))
Comment on attachment 176869 [details] Patch This seems reasonable. A little scary, but reasonable. I agree with Adam that putting adoptPtr on the same line as new is better. It might complicate a couple callers as you couldn't use a raw pointer for a local though.
Committed r136288: <http://trac.webkit.org/changeset/136288>