MissCachedInterventionResult metric should not be counted for resources provided in data url form. |
|||||||
Issue descriptionMissCachedInterventionResult metric should not be counted for resources provided in data url form.
,
Jul 1 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e2f303eb3f571bd2e05de89de3aa08f15437e490 commit e2f303eb3f571bd2e05de89de3aa08f15437e490 Author: toyoshim <toyoshim@chromium.org> Date: Fri Jul 01 14:16:25 2016 WebFonts intervention: exclude data url from the intervention metrics When WebFont resource is provided by data url, do not trigger the intervention, and do not report MissCachedInterventionResult. BUG= 625063 Review-Url: https://codereview.chromium.org/2109353003 Cr-Commit-Position: refs/heads/master@{#403450} [modify] https://crrev.com/e2f303eb3f571bd2e05de89de3aa08f15437e490/third_party/WebKit/Source/core/css/RemoteFontFaceSource.cpp [modify] https://crrev.com/e2f303eb3f571bd2e05de89de3aa08f15437e490/third_party/WebKit/Source/core/css/RemoteFontFaceSource.h
,
Jul 4 2016
,
Jul 4 2016
Your change meets the bar and is auto-approved for M53 (branch: 2785)
,
Jul 5 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/4e372fbc5ca85bf9a1ad8b9c896919572ffd315f commit 4e372fbc5ca85bf9a1ad8b9c896919572ffd315f Author: Takashi Toyoshima <toyoshim@chromium.org> Date: Tue Jul 05 05:55:32 2016 WebFonts intervention: exclude data url from the intervention metrics When WebFont resource is provided by data url, do not trigger the intervention, and do not report MissCachedInterventionResult. BUG= 625063 Review-Url: https://codereview.chromium.org/2109353003 Cr-Commit-Position: refs/heads/master@{#403450} (cherry picked from commit e2f303eb3f571bd2e05de89de3aa08f15437e490) TBR=kinuko@chromium.org Review URL: https://codereview.chromium.org/2121033003 . Cr-Commit-Position: refs/branch-heads/2785@{#16} Cr-Branched-From: 68623971be0cfc492a2cb0427d7f478e7b214c24-refs/heads/master@{#403382} [modify] https://crrev.com/4e372fbc5ca85bf9a1ad8b9c896919572ffd315f/third_party/WebKit/Source/core/css/RemoteFontFaceSource.cpp [modify] https://crrev.com/4e372fbc5ca85bf9a1ad8b9c896919572ffd315f/third_party/WebKit/Source/core/css/RemoteFontFaceSource.h
,
Jul 5 2016
,
Jul 7 2016
toyoshim@ : Could you please let us know the steps to verify if it can be done from TE end.
,
Jul 7 2016
This fix doesn't change user facing behaviors, but changes how to correct histogram data for us, and minor bahaviors behind a field trial expwriment.
,
Jul 8 2016
toyoshim@ : Thank you for the update. |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by sheriffbot@chromium.org
, Jul 1 2016