The LayoutMilestones didLayout callback should only fire for the main frame since it is a WebView/WebCore::Page-level concept.
Created attachment 208042 [details] Patch
Comment on attachment 208042 [details] Patch Sam and I talked about this, and we want to take a slightly different approach here.
Created attachment 208225 [details] Patch
Comment on attachment 208225 [details] Patch Attachment 208225 [details] did not pass qt-ews (qt): Output: http://webkit-queues.appspot.com/results/1418015
Comment on attachment 208225 [details] Patch Attachment 208225 [details] did not pass qt-wk2-ews (qt-wk2): Output: http://webkit-queues.appspot.com/results/1414060
Created attachment 208228 [details] Patch
Comment on attachment 208228 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=208228&action=review > Source/WebCore/loader/FrameLoader.cpp:2286 > +#ifndef NDEBUG You should use !ASSERT_DISABLED here instead.
Thanks Sam! http://trac.webkit.org/changeset/153958