New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 652982 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit 29 days ago
Closed: Oct 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug-Regression

Blocking:
issue 649837



Sign in to add a comment

webkit_tests(http/tests/inspector/persistence/persistence-go-to-file-dialog.html) is flaky

Project Member Reported by horo@chromium.org, Oct 5 2016

Issue description

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=http%2Ftests%2Finspector%2Fpersistence%2Fpersistence-go-to-file-dialog.html%20&showLargeExpectations=true

https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Trusty/18244/layout-test-results/results.html

https://storage.googleapis.com/chromium-layout-test-archives/WebKit_Linux_Trusty/18244/layout-test-results/http/tests/inspector/persistence/persistence-go-to-file-dialog-diff.txt

--- /b/rr/tmp19ThIe/w/layout-test-results/http/tests/inspector/persistence/persistence-go-to-file-dialog-expected.txt
+++ /b/rr/tmp19ThIe/w/layout-test-results/http/tests/inspector/persistence/persistence-go-to-file-dialog-actual.txt
@@ -6,11 +6,11 @@
 Running: goToSourceDialogBeforeBinding
 http://127.0.0.1:8000/inspector/persistence/persistence-go-to-file-dialog.html
 http://127.0.0.1:8000/inspector/inspector-test.js
+http://127.0.0.1:8000/inspector/persistence/persistence-test.js
+http://127.0.0.1:8000/inspector/persistence/resources/foo.js
 http://127.0.0.1:8000/inspector/debugger-test.js
 http://127.0.0.1:8000/inspector/workspace-test.js
 http://127.0.0.1:8000/inspector/isolated-filesystem-test.js
-http://127.0.0.1:8000/inspector/persistence/persistence-test.js
-http://127.0.0.1:8000/inspector/persistence/resources/foo.js
 file:///var/www/inspector/persistence/resources/foo.js
 
 Running: addFileSystemMapping
@@ -18,9 +18,9 @@
 Running: goToSourceAfterBinding
 http://127.0.0.1:8000/inspector/persistence/persistence-go-to-file-dialog.html
 http://127.0.0.1:8000/inspector/inspector-test.js
+http://127.0.0.1:8000/inspector/persistence/persistence-test.js
+http://127.0.0.1:8000/inspector/persistence/resources/foo.js
 http://127.0.0.1:8000/inspector/debugger-test.js
 http://127.0.0.1:8000/inspector/workspace-test.js
 http://127.0.0.1:8000/inspector/isolated-filesystem-test.js
-http://127.0.0.1:8000/inspector/persistence/persistence-test.js
-http://127.0.0.1:8000/inspector/persistence/resources/foo.js
 
 

Comment 1 by horo@chromium.org, Oct 5 2016

Components: Platform>DevTools Platform>DevTools>Authoring
Owner: lushnikov@chromium.org
Status: Assigned (was: Available)
lushnikov@
Please fix this.

Comment 2 by horo@chromium.org, Oct 5 2016

Blocking: 649837

Comment 3 by horo@chromium.org, Oct 5 2016

Labels: -Sheriff-Chromium
Project Member

Comment 4 by bugdroid1@chromium.org, Oct 5 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ce3264f5263919e2d7e1523533aeb95848c937bc

commit ce3264f5263919e2d7e1523533aeb95848c937bc
Author: Tsuyoshi Horo <horo@chromium.org>
Date: Wed Oct 05 10:38:24 2016

Add http/tests/inspector/persistence/persistence-go-to-file-dialog.html to TestExpectations

BUG= 652982 
TBR=lushnikov@chromium.org

Review URL: https://codereview.chromium.org/2395793002 .

Cr-Commit-Position: refs/heads/master@{#423125}

[modify] https://crrev.com/ce3264f5263919e2d7e1523533aeb95848c937bc/third_party/WebKit/LayoutTests/TestExpectations

Status: Started (was: Assigned)
Started: https://codereview.chromium.org/2398773002/

Status: Fixed (was: Started)
Project Member

Comment 8 by bugdroid1@chromium.org, Oct 27 2016

Labels: merge-merged-2840
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ce3264f5263919e2d7e1523533aeb95848c937bc

commit ce3264f5263919e2d7e1523533aeb95848c937bc
Author: Tsuyoshi Horo <horo@chromium.org>
Date: Wed Oct 05 10:38:24 2016

Add http/tests/inspector/persistence/persistence-go-to-file-dialog.html to TestExpectations

BUG= 652982 
TBR=lushnikov@chromium.org

Review URL: https://codereview.chromium.org/2395793002 .

Cr-Commit-Position: refs/heads/master@{#423125}

[modify] https://crrev.com/ce3264f5263919e2d7e1523533aeb95848c937bc/third_party/WebKit/LayoutTests/TestExpectations

Comment 10 by dimu@google.com, Nov 4 2016

Labels: -merge-merged-2840
[Automated comment] removing mislabelled merge-merged-2840

Sign in to add a comment