Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(52)

Issue 2769183003: Lower the webfont loading priority only if the user is in V2 field trial (Closed)

Created:
3 years, 9 months ago by tbansal1
Modified:
3 years, 9 months ago
Reviewers:
CC:
chromium-reviews, blink-reviews-css, dglazkov+blink, apavlov+blink_chromium.org, darktears, blink-reviews, rwlbuis
Target Ref:
refs/branch-heads/3029
Project:
chromium
Visibility:
Public.

Description

Lower the webfont loading priority only if the user is in V2 field trial Currently, the loading priority of web fonts is changed to Low if the user is on 2G network (regardless of if Chrome is a part of WebFontsInterventionV2 field trial or not). This CL adds an additional constraint so that the loading priority is changed only if Chrome is part of V2 field trial. This makes it possible to quantify the effect of changing the loading priority as part of the WebFontsInterventionV2 field trial. BUG=665504 Review-Url: https://codereview.chromium.org/2763523002 Cr-Commit-Position: refs/heads/master@{#458681} (cherry picked from commit fd4fb29de4630be62b296e01987d7bd09d0e4a6e) Review-Url: https://codereview.chromium.org/2769183003 . Cr-Commit-Position: refs/branch-heads/3029@{#393} Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471} Committed: https://chromium.googlesource.com/chromium/src/+/d9e025babf3ed584560c602f7a1adbd7a9d5f8c2

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -8 lines) Patch
M third_party/WebKit/Source/core/css/RemoteFontFaceSource.cpp View 2 chunks +10 lines, -8 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
tbansal1
3 years, 9 months ago (2017-03-23 20:26:26 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
d9e025babf3ed584560c602f7a1adbd7a9d5f8c2.

Powered by Google App Engine
This is Rietveld 408576698