Summary: | Resource Load Statistics: Make sure WebResourceLoadStatisticsStore::mergeWithDataFromDecoder() can ingest older plist versions and not reset the database | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | John Wilander <wilander> | ||||||
Component: | WebKit Misc. | Assignee: | John Wilander <wilander> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bfulgham, cdumez, commit-queue, dbates, ews-watchlist, japhet, jlewis3, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
John Wilander
2018-02-14 14:53:00 PST
Created attachment 333849 [details]
Patch
Comment on attachment 333849 [details]
Patch
Looks good. r=me.
Comment on attachment 333849 [details]
Patch
Thanks, Brent!
Comment on attachment 333849 [details] Patch Clearing flags on attachment: 333849 Committed r228495: <https://trac.webkit.org/changeset/228495> All reviewed patches have been landed. Closing bug. This broke performance tests with unexpected logging: https://build.webkit.org/builders/Apple%20Sierra%20Release%20WK2%20%28Perf%29/builds/236/steps/perf-test/logs/stdio This caused the Apple Sierra Release WK2 (Perf) testers to fail 183 perf tests consistently: https://build.webkit.org/builders/Apple%20Sierra%20Release%20WK2%20%28Perf%29/builds/231 https://build.webkit.org/builders/Apple%20Sierra%20Release%20WK2%20%28Perf%29/builds/231/steps/perf-test/logs/stdio Reverted r228495 for reason: This caused mulitple perf tests to fail consistently. Committed r228514: <https://trac.webkit.org/changeset/228514> (In reply to Alexey Proskuryakov from comment #7) > This broke performance tests with unexpected logging: > https://build.webkit.org/builders/ > Apple%20Sierra%20Release%20WK2%20%28Perf%29/builds/236/steps/perf-test/logs/ > stdio I have no idea why perf tests are ingesting statistics of model 1. Either this is some special statistics plist, a super old test, or a workaround for something else. One way to fix this is to avoid model 1. Created attachment 333934 [details]
Patch for landing
Removed logging for model version upgrades all together. It's mostly the early return for skipped ingestion that we need to log. The other one was intended to help with bug investigations since it would tell us if an upgrade had taken place. Comment on attachment 333934 [details] Patch for landing Clearing flags on attachment: 333934 Committed r228532: <https://trac.webkit.org/changeset/228532> All reviewed patches have been landed. Closing bug. |