Bug 70985 – inspector/storage-panel-dom-storage.html has results platform dependant?

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Julien Chaffraix]

The test starting failing on the linux (and maybe win7) bots. Here is the diff:

--- /mnt/data/b/build/slave/Webkit_Linux/build/layout-test-results/inspector/storage-panel-dom-storage-expected.txt 
+++ /mnt/data/b/build/slave/Webkit_Linux/build/layout-test-results/inspector/storage-panel-dom-storage-actual.txt 
@@ -4,8 +4,8 @@
 Did show: Local storage
 Did show: Session storage
 Local storage content: 
-KeyValueresource-history{}a4=value4, a0=value0, a2=value2, a7=value7, a9=value9, a5=value5, a8=value8, a1=value1, a3=value3, a6=value6, 
+KeyValuea0=value0, a6=value6, a8=value8, a4=value4, a2=value2, a1=value1, resource-history{}a3=value3, a9=value9, a7=value7, a5=value5, 
 Session storage content: 
-KeyValueb4=value14, b3=value13, b0=value10, b9=value19, b5=value15, b6=value16, b7=value17, b8=value18, b1=value11, b2=value12, 
+KeyValueb5=value15, b4=value14, b3=value13, b6=value16, b7=value17, b0=value10, b9=value19, b8=value18, b2=value12, b1=value11, 
 DONE

I don't know if the new result make any sense so I will just mark the test as TEXT.

[Yury Semikhatsky]

Looks like a bug in the test itself, Ilya is looking into it.

[Csaba Osztrogonác]

Qt platform has same result as Julien mentioned.

[Ilya Tikhonovsky]

Created attachment 112646 [details]
Patch

[Yury Semikhatsky]

Comment on attachment 112646 [details]
Patch

Clearing flags on attachment: 112646

Committed r98538: <http://trac.webkit.org/changeset/98538>

[Yury Semikhatsky]

All reviewed patches have been landed.  Closing bug.

[Csaba Osztrogonác]

Thanks for the quick fix.