RenderBoxModelObject's continuation() does a hash lookup so calling it repeatedly is not super cool.
Created attachment 212468 [details] Patch
Comment on attachment 212468 [details] Patch Attachment 212468 [details] did not pass qt-ews (qt): Output: http://webkit-queues.appspot.com/results/1908496
Comment on attachment 212468 [details] Patch Attachment 212468 [details] did not pass qt-wk2-ews (qt-wk2): Output: http://webkit-queues.appspot.com/results/2004627
Comment on attachment 212468 [details] Patch Attachment 212468 [details] did not pass efl-wk2-ews (efl-wk2): Output: http://webkit-queues.appspot.com/results/2110241
Comment on attachment 212468 [details] Patch Attachment 212468 [details] did not pass efl-ews (efl): Output: http://webkit-queues.appspot.com/results/2004629
Comment on attachment 212468 [details] Patch Attachment 212468 [details] did not pass gtk-ews (gtk): Output: http://webkit-queues.appspot.com/results/1960269
Comment on attachment 212468 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=212468&action=review r=me, plz fix build -- see below > Source/WebCore/rendering/RenderInline.cpp:1439 > + continuation->addFocusRingRects(rects, flooredLayoutPoint(additionalOffset + toRenderBox(continuation())->location() - containingBlock()->location()), paintContainer); Nix the parens on 'continuation' here.
Comment on attachment 212468 [details] Patch Attachment 212468 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/1908503
Created attachment 212471 [details] Patch for laundry
Comment on attachment 212471 [details] Patch for laundry Clearing flags on attachment: 212471 Committed r156334: <http://trac.webkit.org/changeset/156334>
All reviewed patches have been landed. Closing bug.