Bug 114209

Summary: Flaky Test: http/tests/inspector/appcache/appcache-manifest-with-non-existing-file.html
Product: WebKit Reporter: WebKit Commit Bot <commit-queue>
Component: Tools / TestsAssignee: Alexey Proskuryakov <ap>
Status: RESOLVED FIXED    
Severity: Normal CC: joepeck, vsevik
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 50856    
Attachments:
Description Flags
Archive of layout-test-results from webkit-cq-03
none
Archive of layout-test-results from webkit-cq-03
none
Archive of layout-test-results from webkit-cq-03
none
Archive of layout-test-results from webkit-cq-01
none
Archive of layout-test-results from webkit-cq-03
none
Archive of layout-test-results from webkit-cq-02
none
Archive of layout-test-results from webkit-cq-02
none
Archive of layout-test-results from webkit-cq-03
none
skip the test joepeck: review+

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>.