User Agent Intervention: WebFonts loading spams the console |
||||||
Issue descriptionChrome Version: 63.0.3208.0 (Official Build) canary (64-bit) OS: OSX 10.12.6 What steps will reproduce the problem? (1) Do some local development work on a Polymer project. (2) Open the JS console. What is the expected result? No console spam. What happens instead? I frequently see ~1000 lines of console messages that say "Slow network is detected. Fallback font will be used ..." Could you please limit the console spam to <<1000 lines -- ideally one or two lines, at most? Also, it seems to me like this intervention is over-triggering. I'm reproducing this on a pretty darned fast network, doing local development ;-)
,
Sep 12 2017
For more context, we are currently triggering intervention when we think that connection is slower than 3G which maps to an RTT of >= ~275 msec.
,
Sep 14 2017
I just filed feedback after hitting this again during local development. How do I get a URL to the report? I landed at https://support.google.com/chrome/answer/186850?visit_id=1-636409725634835979-1429306010&p=feedback_confirmation&rd=1 -- is the visit_id sufficient to find the feedback report?
,
Sep 14 2017
Removed RVG label after removing some of the comments. Issue 765378 tracks the overtriggering bug.
,
Sep 15 2017
Thank you for investigation. So, unexpected intervention triggering are discussed at Issue 765378 , but I have another question about the ~1000 lines warnings. Do you really have ~1000 WebFonts entries in the page? I'm wondering if I could omit the repeated warnings on the same WebFont URL. But if URL are different, I think it's still useful for developers who open the console.
,
Sep 15 2017
A recent case that I saw has 585 lines, attached. I believe that I saw closer to ~1000 in a previous case; but regardless, it's quite a lot ;-)
,
Sep 19 2017
Looks like we're reporting this for every @font-face rule, regardless of whether it's used or not. Maybe we should report it when the font started to load, in RemoteFontFaceSource::BeginLoadIfNeeded().
,
Sep 19 2017
Thanks! I will adopt what ksakamoto suggests. That would help to minimize the number of warnings.
,
Sep 20 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/667294139c3c90290ad5297f5bb90cfa82f0d2cd commit 667294139c3c90290ad5297f5bb90cfa82f0d2cd Author: Takashi Toyoshima <toyoshim@chromium.org> Date: Wed Sep 20 00:31:19 2017 WebFonts intervention: show a warning on actually making a request Before this change, warnings are shown for all font-face rules. This will spam the console if specified css files contain many unused rules. With this change, warnings are shown on making actual font loading requests, or when fonts are requested to use in the page rendering. Bug: 764204 Change-Id: I6b3e6627a5c6d5b92e36408ef5ff0951e7903bc7 Reviewed-on: https://chromium-review.googlesource.com/672178 Reviewed-by: Kunihiko Sakamoto <ksakamoto@chromium.org> Reviewed-by: Kinuko Yasuda (slow) <kinuko@chromium.org> Commit-Queue: Takashi Toyoshima <toyoshim@chromium.org> Cr-Commit-Position: refs/heads/master@{#503006} [modify] https://crrev.com/667294139c3c90290ad5297f5bb90cfa82f0d2cd/third_party/WebKit/Source/core/css/RemoteFontFaceSource.cpp
,
Sep 20 2017
One possible fix was landed. Warnings should be shown only for actually used WebFonts. I hope this can improve users' experience. Ilya, could you check if this change helps your case? Intervention false trigger would not happen on POSIX ports as far as I heard from tbansal. So you can check it on HEAD/Canary for Mac or Windows, I think. If it does not work, please reopen this bug again.
,
Sep 20 2017
Thanks! I'll test this once the Canary version updates to include the fix.
,
Sep 23 2017
I can confirm that the mitigation is working well. I now only see a single line logged to the console, which is much more manageable.
,
Sep 23 2017
,
Dec 13 2017
|
||||||
►
Sign in to add a comment |
||||||
Comment 1 by tbansal@chromium.org
, Sep 12 2017