Bug 114209 - Flaky Test: http/tests/inspector/appcache/appcache-manifest-with-non-existing-file.html
Summary: Flaky Test: http/tests/inspector/appcache/appcache-manifest-with-non-existing...
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: Alexey Proskuryakov
URL:
Keywords:
Depends on:
Blocks: 50856
  Show dependency treegraph
 
Reported: 2013-04-08 13:03 PDT by WebKit Commit Bot
Modified: 2013-09-13 13:41 PDT (History)
2 users (show)

See Also:


Attachments
Archive of layout-test-results from webkit-cq-03 (547.77 KB, application/zip)
2013-04-08 13:03 PDT, WebKit Commit Bot
no flags Details
Archive of layout-test-results from webkit-cq-03 (583.53 KB, application/zip)
2013-04-09 06:07 PDT, WebKit Commit Bot
no flags Details
Archive of layout-test-results from webkit-cq-03 (547.33 KB, application/zip)
2013-04-10 09:24 PDT, WebKit Commit Bot
no flags Details
Archive of layout-test-results from webkit-cq-01 (549.06 KB, application/zip)
2013-04-10 16:51 PDT, WebKit Commit Bot
no flags Details
Archive of layout-test-results from webkit-cq-03 (543.46 KB, application/zip)
2013-04-19 08:58 PDT, WebKit Commit Bot
no flags Details
Archive of layout-test-results from webkit-cq-02 (546.04 KB, application/zip)
2013-05-06 00:30 PDT, WebKit Commit Bot
no flags Details
Archive of layout-test-results from webkit-cq-02 (581.12 KB, application/zip)
2013-05-06 13:53 PDT, WebKit Commit Bot
no flags Details
Archive of layout-test-results from webkit-cq-03 (563.32 KB, application/zip)
2013-05-22 11:36 PDT, WebKit Commit Bot
no flags Details
skip the test (2.71 KB, patch)
2013-09-13 13:36 PDT, Alexey Proskuryakov
joepeck: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description WebKit Commit Bot 2013-04-08 13:03:04 PDT
This is an automatically generated bug from the commit-queue.
http/tests/inspector/appcache/appcache-manifest-with-non-existing-file.html has been flaky on the commit-queue.

http/tests/inspector/appcache/appcache-manifest-with-non-existing-file.html was authored by vsevik@chromium.org.
http://trac.webkit.org/browser/trunk/LayoutTests/http/tests/inspector/appcache/appcache-manifest-with-non-existing-file.html

The commit-queue just saw http/tests/inspector/appcache/appcache-manifest-with-non-existing-file.html flake (text diff) while processing attachment 196863 [details] on bug 114172.
Bot: webkit-cq-03  Port: <class 'webkitpy.common.config.ports.MacPort'>  Platform: Mac OS X 10.8.3

The bots will update this with information from each new failure.

If you believe this bug to be fixed or invalid, feel free to close.  The bots will re-open if the flake re-occurs.

If you would like to track this test fix with another bug, please close this bug as a duplicate.  The bots will follow the duplicate chain when making future comments.
Comment 1 WebKit Commit Bot 2013-04-08 13:03:06 PDT
Created attachment 196914 [details]
Archive of layout-test-results from webkit-cq-03
Comment 2 WebKit Commit Bot 2013-04-09 06:07:21 PDT
The commit-queue just saw http/tests/inspector/appcache/appcache-manifest-with-non-existing-file.html flake (text diff) while processing attachment 197031 [details] on bug 114269.
Bot: webkit-cq-03  Port: <class 'webkitpy.common.config.ports.MacPort'>  Platform: Mac OS X 10.8.3
Comment 3 WebKit Commit Bot 2013-04-09 06:07:23 PDT
Created attachment 197037 [details]
Archive of layout-test-results from webkit-cq-03
Comment 4 WebKit Commit Bot 2013-04-10 09:24:25 PDT
The commit-queue just saw http/tests/inspector/appcache/appcache-manifest-with-non-existing-file.html flake (text diff) while processing attachment 197237 [details] on bug 114346.
Bot: webkit-cq-03  Port: <class 'webkitpy.common.config.ports.MacPort'>  Platform: Mac OS X 10.8.3
Comment 5 WebKit Commit Bot 2013-04-10 09:24:26 PDT
Created attachment 197302 [details]
Archive of layout-test-results from webkit-cq-03
Comment 6 WebKit Commit Bot 2013-04-10 16:51:01 PDT
The commit-queue just saw http/tests/inspector/appcache/appcache-manifest-with-non-existing-file.html flake (text diff) while processing attachment 197414 [details] on bug 114390.
Bot: webkit-cq-01  Port: <class 'webkitpy.common.config.ports.MacPort'>  Platform: Mac OS X 10.8.3
Comment 7 WebKit Commit Bot 2013-04-10 16:51:03 PDT
Created attachment 197446 [details]
Archive of layout-test-results from webkit-cq-01
Comment 8 WebKit Commit Bot 2013-04-19 08:58:48 PDT
The commit-queue just saw http/tests/inspector/appcache/appcache-manifest-with-non-existing-file.html flake (text diff) while processing attachment 198850 [details] on bug 109588.
Bot: webkit-cq-03  Port: <class 'webkitpy.common.config.ports.MacPort'>  Platform: Mac OS X 10.8.3
Comment 9 WebKit Commit Bot 2013-04-19 08:58:49 PDT
Created attachment 198885 [details]
Archive of layout-test-results from webkit-cq-03
Comment 10 WebKit Commit Bot 2013-05-06 00:30:27 PDT
The commit-queue just saw http/tests/inspector/appcache/appcache-manifest-with-non-existing-file.html flake (text diff) while processing attachment 200589 [details] on bug 115611.
Bot: webkit-cq-02  Port: <class 'webkitpy.common.config.ports.MacPort'>  Platform: Mac OS X 10.8.3
Comment 11 WebKit Commit Bot 2013-05-06 00:30:28 PDT
Created attachment 200636 [details]
Archive of layout-test-results from webkit-cq-02
Comment 12 WebKit Commit Bot 2013-05-06 13:53:51 PDT
The commit-queue just saw http/tests/inspector/appcache/appcache-manifest-with-non-existing-file.html flake (text diff) while processing attachment 200768 [details] on bug 113808.
Bot: webkit-cq-02  Port: <class 'webkitpy.common.config.ports.MacPort'>  Platform: Mac OS X 10.8.3
Comment 13 WebKit Commit Bot 2013-05-06 13:53:53 PDT
Created attachment 200801 [details]
Archive of layout-test-results from webkit-cq-02
Comment 14 WebKit Commit Bot 2013-05-22 11:36:05 PDT
The commit-queue just saw http/tests/inspector/appcache/appcache-manifest-with-non-existing-file.html flake (text diff) while processing attachment 202573 [details] on bug 116581.
Bot: webkit-cq-03  Port: <class 'webkitpy.common.config.ports.MacPort'>  Platform: Mac OS X 10.8.3
Comment 15 WebKit Commit Bot 2013-05-22 11:36:07 PDT
Created attachment 202605 [details]
Archive of layout-test-results from webkit-cq-03
Comment 16 Alexey Proskuryakov 2013-09-13 13:36:15 PDT
Created attachment 211584 [details]
skip the test

This functionality is hopelessly broken in deprecated web inspector. The test intends to check application cache events in the case when the cache fails to download, but front-end doesn't track cache status at all until the document has an associated cache. So when the test passes, it's accidental.

ApplicationCacheModel._frameManifestUpdated just bails out if manifest URL is empty, so status remains 0 even while download is in progress.

Initially, the test correctly waited for the state to become DOWNLOADING, and then to become UNCACHED again. But <http://trac.webkit.org/changeset/100454> dumbed it down, presumably to cover a bug introduced in that revision (there is no explanation in ChangeLog, which pretends that the file was moved unchanged).

Keeping the test around in case it could serve as a reminder to write a similar one for new front-end.
Comment 17 Joseph Pecoraro 2013-09-13 13:39:09 PDT
Comment on attachment 211584 [details]
skip the test

View in context: https://bugs.webkit.org/attachment.cgi?id=211584&action=review

r=me

> LayoutTests/ChangeLog:13
> +        Removed from the lists, as the test is now skipepd globally.

Typo: "skipepd"
Comment 18 Alexey Proskuryakov 2013-09-13 13:41:19 PDT
Committed <http://trac.webkit.org/r155720>.