Bug 70985 - inspector/storage-panel-dom-storage.html has results platform dependant?
Summary: inspector/storage-panel-dom-storage.html has results platform dependant?
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ilya Tikhonovsky
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-10-26 18:08 PDT by Julien Chaffraix
Modified: 2011-10-27 02:21 PDT (History)
4 users (show)

See Also:


Attachments
Patch (4.17 KB, patch)
2011-10-27 00:38 PDT, Ilya Tikhonovsky
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Julien Chaffraix 2011-10-26 18:08:41 PDT
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.
Comment 1 Yury Semikhatsky 2011-10-26 23:36:15 PDT
Looks like a bug in the test itself, Ilya is looking into it.
Comment 2 Csaba Osztrogonác 2011-10-26 23:52:01 PDT
Qt platform has same result as Julien mentioned.
Comment 3 Ilya Tikhonovsky 2011-10-27 00:38:50 PDT
Created attachment 112646 [details]
Patch
Comment 4 Yury Semikhatsky 2011-10-27 00:53:58 PDT
Comment on attachment 112646 [details]
Patch

Clearing flags on attachment: 112646

Committed r98538: <http://trac.webkit.org/changeset/98538>
Comment 5 Yury Semikhatsky 2011-10-27 00:54:06 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Csaba Osztrogonác 2011-10-27 02:21:25 PDT
Thanks for the quick fix.