| Summary: | [GTK][WPE][Fetch API] Stale-while-revalidate is not enabled | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Diego Pino <dpino> | ||||||||||
| Component: | WebKitGTK | Assignee: | Rob Buis <rbuis> | ||||||||||
| Status: | RESOLVED FIXED | ||||||||||||
| Severity: | Normal | CC: | annulen, bugs-noreply, clopez, commit-queue, ews-watchlist, fred.wang, gyuyoung.kim, rbuis, ryuan.choi, sergio | ||||||||||
| Priority: | P2 | ||||||||||||
| Version: | WebKit Nightly Build | ||||||||||||
| Hardware: | Unspecified | ||||||||||||
| OS: | Unspecified | ||||||||||||
| Attachments: |
|
||||||||||||
|
Description
Diego Pino
2020-01-17 08:35:19 PST
Created attachment 388672 [details]
Patch
Comment on attachment 388672 [details] Patch Rejecting attachment 388672 [details] from commit-queue. New failing tests: editing/spelling/spellcheck-async-remove-frame.html Full output: https://webkit-queues.webkit.org/results/13311067 Created attachment 388703 [details]
Archive of layout-test-results from webkit-cq-03 for mac-mojave
The attached test failures were seen while running run-webkit-tests on the commit-queue.
Bot: webkit-cq-03 Port: mac-mojave Platform: Mac OS X 10.14.6
Comment on attachment 388672 [details] Patch Clearing flags on attachment: 388672 Committed r255078: <https://trac.webkit.org/changeset/255078> All reviewed patches have been landed. Closing bug. Re-opening and re-titling to make more clear why this tests fail: Stale-while-revalidate is not enabled for GTK/WPE Created attachment 397944 [details]
Patch
Comment on attachment 397944 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=397944&action=review > LayoutTests/platform/wpe/TestExpectations:-472 > -# Stale-while-revalidate is not enabled for WPE > -webkit.org/b/206416 imported/w3c/web-platform-tests/fetch/stale-while-revalidate [ Skip ] I don't see this patch enabling the feature for WPE at Source/cmake/OptionsWPE.cmake It seems ENABLE_NETWORK_CACHE_SPECULATIVE_REVALIDATION its also not enabled for WPE, not sure if needed. And note that the GTK and WPE EWS don't run layout tests, only test to build. Someone has to manually verify that this tests now pass after enabling the feature Created attachment 397957 [details]
Patch
Comment on attachment 397944 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=397944&action=review >> LayoutTests/platform/wpe/TestExpectations:-472 >> -webkit.org/b/206416 imported/w3c/web-platform-tests/fetch/stale-while-revalidate [ Skip ] > > I don't see this patch enabling the feature for WPE at Source/cmake/OptionsWPE.cmake > It seems ENABLE_NETWORK_CACHE_SPECULATIVE_REVALIDATION its also not enabled for WPE, not sure if needed. > And note that the GTK and WPE EWS don't run layout tests, only test to build. Someone has to manually verify that this tests now pass after enabling the feature Right, I was confused because I did not see ENABLE_NETWORK_CACHE_SPECULATIVE_REVALIDATION enabled for WPE, however the s-w-r should be independent with its own build flag, so I now added it. I verified locally GTK layout tests work, however I do not usually build WPE. Are you willing to build it and run the s-w-r tests? (In reply to Rob Buis from comment #10) > Are you willing to build it and run the s-w-r tests? Tested. The 6 tests pass here. (In reply to Rob Buis from comment #10) > Right, I was confused because I did not see > ENABLE_NETWORK_CACHE_SPECULATIVE_REVALIDATION enabled for WPE I guess nobody cared to enable it for WPE so far. Any reason to not enable it for WPE? If it works for GTK, it should work also for WPE. Both ports share the same network stack (In reply to Carlos Alberto Lopez Perez from comment #12) > (In reply to Rob Buis from comment #10) > > Right, I was confused because I did not see > > ENABLE_NETWORK_CACHE_SPECULATIVE_REVALIDATION enabled for WPE > > I guess nobody cared to enable it for WPE so far. > Any reason to not enable it for WPE? If it works for GTK, it should work > also for WPE. Both ports share the same network stack I can't think of any reason not to enable it for WPE. Committed r261161: <https://trac.webkit.org/changeset/261161> All reviewed patches have been landed. Closing bug and clearing flags on attachment 397957 [details]. |