Store a visible rect in GraphicsLayers, and optionally dump it in layerTreeAsText
Created attachment 167890 [details] Patch
Comment on attachment 167890 [details] Patch Attachment 167890 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/14250146
Comment on attachment 167890 [details] Patch Attachment 167890 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/14215982
Comment on attachment 167890 [details] Patch Attachment 167890 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/14245339
Comment on attachment 167890 [details] Patch Attachment 167890 [details] did not pass efl-ews (efl): Output: http://queues.webkit.org/results/14250158
Comment on attachment 167890 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=167890&action=review Looks good, but more things need to build. > Source/WebCore/platform/graphics/GraphicsLayer.cpp:695 > + WTF::String output = layer->layerTreeAsText(LayerTreeAsTextDebug | LayerTreeAsTextIncludeVisibleRects); Not new, but the WTF:: here is not necessary. > Source/WebCore/platform/graphics/GraphicsLayer.h:439 > + static void writeIndent(TextStream&, int /*indent*/); There is no reason to comment out the word indent here.
Created attachment 168016 [details] Patch
Created attachment 168017 [details] Patch
Comment on attachment 168017 [details] Patch Attachment 168017 [details] did not pass efl-ews (efl): Output: http://queues.webkit.org/results/14254221
Created attachment 168021 [details] Patch
Comment on attachment 168021 [details] Patch Attachment 168021 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/14245581
Comment on attachment 168021 [details] Patch Attachment 168021 [details] did not pass gtk-ews (gtk): Output: http://queues.webkit.org/results/14260089
http://trac.webkit.org/changeset/130927 This will break the Windows build with a link error, which I'll use to get the right mangled symbol to export.