Make the dumped display list representation a bit more compact
authorsimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 24 Jan 2016 20:39:49 +0000 (20:39 +0000)
committersimon.fraser@apple.com <simon.fraser@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 24 Jan 2016 20:39:49 +0000 (20:39 +0000)
https://bugs.webkit.org/show_bug.cgi?id=153409

Reviewed by Zalan Bujtas.
Source/WebCore:

Don't make a group for every display list item.

* platform/graphics/displaylists/DisplayList.cpp:
(WebCore::DisplayList::DisplayList::asText):

LayoutTests:

Updated results.

* displaylists/extent-includes-shadow-expected.txt:
* displaylists/extent-includes-transforms-expected.txt:
* displaylists/layer-dispay-list-expected.txt:
* displaylists/replay-skip-clipped-rect-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@195516 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/displaylists/extent-includes-shadow-expected.txt
LayoutTests/displaylists/extent-includes-transforms-expected.txt
LayoutTests/displaylists/layer-dispay-list-expected.txt
LayoutTests/displaylists/replay-skip-clipped-rect-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/displaylists/DisplayList.cpp

index d1cf9af..5062f1d 100644 (file)
@@ -1,5 +1,19 @@
 2016-01-24  Simon Fraser  <simon.fraser@apple.com>
 
 2016-01-24  Simon Fraser  <simon.fraser@apple.com>
 
+        Make the dumped display list representation a bit more compact
+        https://bugs.webkit.org/show_bug.cgi?id=153409
+
+        Reviewed by Zalan Bujtas.
+        
+        Updated results.
+
+        * displaylists/extent-includes-shadow-expected.txt:
+        * displaylists/extent-includes-transforms-expected.txt:
+        * displaylists/layer-dispay-list-expected.txt:
+        * displaylists/replay-skip-clipped-rect-expected.txt:
+
+2016-01-24  Simon Fraser  <simon.fraser@apple.com>
+
         Add testing for display list replay, and skip clipped-out items on replay
         https://bugs.webkit.org/show_bug.cgi?id=153408
 
         Add testing for display list replay, and skip clipped-out items on replay
         https://bugs.webkit.org/show_bug.cgi?id=153408
 
index 39621a9..51ff9df 100644 (file)
@@ -1,21 +1,16 @@
 
 
-(
-  (translate
-    (x 0.00)
-    (y 0.00)))
-(
-  (save
-    (restore-index 4)))
-(
-  (set-state
-    (change-flags 1024)
-    (shadow-blur 12.00)
-    (shadow-offset width=10 height=20)
-    (shadows-use-legacy-radius 0)))
-(
-  (fill-rect-with-color
-    (extent at (43,50) size 134x137)
-    (rect at (50,50) size 100x100)
-    (color #0000FF)))
-(
-  (restore))
+(translate
+  (x 0.00)
+  (y 0.00))
+(save
+  (restore-index 4))
+(set-state
+  (change-flags 1024)
+  (shadow-blur 12.00)
+  (shadow-offset width=10 height=20)
+  (shadows-use-legacy-radius 0))
+(fill-rect-with-color
+  (extent at (43,50) size 134x137)
+  (rect at (50,50) size 100x100)
+  (color #0000FF))
+(restore)
index 4d6a356..1cbbd21 100644 (file)
@@ -1,24 +1,18 @@
 
 
-(
-  (translate
-    (x 0.00)
-    (y 22.00)))
-(
-  (concatentate-ctm
-    (ctm {m=((0.87,0.50)(-0.50,0.87)) t=(81.70,-18.30)})))
-(
-  (save
-    (restore-index 5)))
-(
-  (set-state
-    (change-flags 1024)
-    (shadow-blur 12.00)
-    (shadow-offset width=10 height=20)
-    (shadows-use-legacy-radius 0)))
-(
-  (fill-rect-with-color
-    (extent at (7.14,0.20) size 184.55x185.65)
-    (rect at (0,0) size 100x100)
-    (color #0000FF)))
-(
-  (restore))
+(translate
+  (x 0.00)
+  (y 22.00))
+(concatentate-ctm
+  (ctm {m=((0.87,0.50)(-0.50,0.87)) t=(81.70,-18.30)}))
+(save
+  (restore-index 5))
+(set-state
+  (change-flags 1024)
+  (shadow-blur 12.00)
+  (shadow-offset width=10 height=20)
+  (shadows-use-legacy-radius 0))
+(fill-rect-with-color
+  (extent at (7.14,0.20) size 184.55x185.65)
+  (rect at (0,0) size 100x100)
+  (color #0000FF))
+(restore)
index 164ec02..32cc71a 100644 (file)
@@ -1,36 +1,29 @@
 
 
-(
-  (translate
-    (x 0.00)
-    (y 0.00)))
-(
-  (fill-rect-with-color
-    (extent at (0,0) size 104x104)
-    (rect at (0,0) size 104x104)
-    (color #008000)))
-(
-  (set-state
-    (change-flags 131456)
-    (fill-color #0000FF)
-    (stroke-style 0)
-    (should-antialias 0)))
-(
-  (draw-rect
-    (extent at (0,0) size 104x2)
-    (rect at (0,0) size 104x2)
-    (border-thickness 1.00)))
-(
-  (draw-rect
-    (extent at (0,102) size 104x2)
-    (rect at (0,102) size 104x2)
-    (border-thickness 1.00)))
-(
-  (draw-rect
-    (extent at (0,0) size 2x104)
-    (rect at (0,0) size 2x104)
-    (border-thickness 1.00)))
-(
-  (draw-rect
-    (extent at (102,0) size 2x104)
-    (rect at (102,0) size 2x104)
-    (border-thickness 1.00)))
+(translate
+  (x 0.00)
+  (y 0.00))
+(fill-rect-with-color
+  (extent at (0,0) size 104x104)
+  (rect at (0,0) size 104x104)
+  (color #008000))
+(set-state
+  (change-flags 131456)
+  (fill-color #0000FF)
+  (stroke-style 0)
+  (should-antialias 0))
+(draw-rect
+  (extent at (0,0) size 104x2)
+  (rect at (0,0) size 104x2)
+  (border-thickness 1.00))
+(draw-rect
+  (extent at (0,102) size 104x2)
+  (rect at (0,102) size 104x2)
+  (border-thickness 1.00))
+(draw-rect
+  (extent at (0,0) size 2x104)
+  (rect at (0,0) size 2x104)
+  (border-thickness 1.00))
+(draw-rect
+  (extent at (102,0) size 2x104)
+  (rect at (102,0) size 2x104)
+  (border-thickness 1.00))
index 888cc47..31914b5 100644 (file)
@@ -1,16 +1,15 @@
 recorded: 
 recorded: 
-(
-  (translate
-    (x 0.00)
-    (y 0.00)))
-(
-  (fill-rect-with-color
-    (extent at (412,0) size 100x100)
-    (rect at (412,0) size 100x100)
-    (color #0000FF)))
+(translate
+  (x 0.00)
+  (y 0.00))
+(fill-rect-with-color
+  (extent at (412,0) size 100x100)
+  (rect at (412,0) size 100x100)
+  (color #0000FF))
 
 replayed: 
 (
 
 replayed: 
 (
-  (translate
-    (x 0.00)
-    (y 0.00)))
+  (clip at (512,0) size 512x200)
+(translate
+  (x 0.00)
+  (y 0.00))
index 7fa71fb..856b2af 100644 (file)
@@ -1,5 +1,17 @@
 2016-01-24  Simon Fraser  <simon.fraser@apple.com>
 
 2016-01-24  Simon Fraser  <simon.fraser@apple.com>
 
+        Make the dumped display list representation a bit more compact
+        https://bugs.webkit.org/show_bug.cgi?id=153409
+
+        Reviewed by Zalan Bujtas.
+
+        Don't make a group for every display list item.
+
+        * platform/graphics/displaylists/DisplayList.cpp:
+        (WebCore::DisplayList::DisplayList::asText):
+
+2016-01-24  Simon Fraser  <simon.fraser@apple.com>
+
         Add testing for display list replay, and skip clipped-out items on replay
         https://bugs.webkit.org/show_bug.cgi?id=153408
 
         Add testing for display list replay, and skip clipped-out items on replay
         https://bugs.webkit.org/show_bug.cgi?id=153408
 
index 3aa5400..6b3d210 100644 (file)
@@ -91,13 +91,9 @@ bool DisplayList::shouldDumpForFlags(AsTextFlags flags, const Item& item)
 String DisplayList::asText(AsTextFlags flags) const
 {
     TextStream stream;
 String DisplayList::asText(AsTextFlags flags) const
 {
     TextStream stream;
-    size_t numItems = m_list.size();
-    for (size_t i = 0; i < numItems; ++i) {
-        const auto& item = m_list[i].get();
+    for (auto& item : m_list) {
         if (!shouldDumpForFlags(flags, item))
             continue;
         if (!shouldDumpForFlags(flags, item))
             continue;
-
-        TextStream::GroupScope scope(stream);
         stream << item;
     }
     return stream.release();
         stream << item;
     }
     return stream.release();