New issue
Advanced search Search tips

Issue 881614 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Nov 27
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Slow network intervention should not be displayed as error level logs

Reported by a...@helloclub.com, Sep 6

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/68.0.3440.106 Safari/537.36

Steps to reproduce the problem:
1. Use developer tools to inspect a page while you're on a "slow" network
2. Observe countless "errors" informing you that a slow network has been detected and that a fallback font will be used.

What is the expected behavior?
That we either don't see these "errors" which are not errors at all, or that they go back to having reduced logging level so that we can filter them out or don't see them by default.

What went wrong?
When developing, we are always on the lookout for errors in the console.  Now, on our "slow" network, we are faced with 3 "errors" in the console every time we refresh the page.

These are not errors, the font display engine is working accordingly as per the specified `font-display` property, and is swapping fonts correctly. Why do develops needs to see these obnoxious messages as errors? They should be verbose or info level warnings at best, and ideally we should be able to turn them off completely because they just create unnecessary noise.

Did this work before? Yes One or two versions back these didn't show up as red errors.

Chrome version: 68.0.3440.106  Channel: n/a
OS Version: OS X 10.11.6
Flash Version: 

Developer tools are there to help developers, not to get in their way or annoy them with irrelevant warnings.
 
Screenshot 2018-09-07 11.07.20.png
50.6 KB View Download
I would also like to add that the official documentation page (https://www.chromestatus.com/feature/5636954674692096) of this feature states:

"Web site owners can opt-out from the intervention by setting CSS font-display to other than 'auto'."

Our font-display property is set to `swap` or `optional`, yet we still keep seeing these errors.
Cc: toyoshim@chromium.org ksakamoto@chromium.org
Components: -Platform>DevTools Blink>Loader
I am going to change the message level from "error" to "warning". The message should not show up when using `swap` or `optional`. cc'ing toyoshim@ and ksakamoto@ for that.
Adam, do you see this error when you are using network throttling on devtools, or do you see it at all times?
Thanks, we see it all the time. We don't use network throttling at all.
We use `font-display` to try to avoid them, but the warnings still show up.

Warnings also show up for 3rd party plugins that use web fonts, which we have no control over though. These warnings can't be filtered out in the console, and area really distracting and cumbersome.
That's bad.

I can confirm in the latest Chrome version (M71), these message show up at Info level. So, that problem is already fixed.

We still need to figure out why these are showing up for non-auto fonts, and on fast connections.
Also, I suspect there's still something wrong with the actual algorithms detecting a slow network, as we often get these messages on our gigabit internet connection.

It could be that the CDN or site the fonts are loaded from is slow, but in that case the warning text is misleading, because your own network connection is fine.
Great to hear it's at least back to info, but too bad we have to wait 3 versions until this is fixed.
Right, the algorithm tries to differentiate between network being slow and server being slow (e.g., in case of hanging GETs, long polling requests, XHRs). However, sometimes it gives false positives. We are looking at ways to improve the accuracy.  Issue 823322  and  Issue  834119  track the work.
You can start Chrome with command line switch --force-effective-connection-type=4G to disable this. Alternatively, in chrome://flags/, set the dropdown in "Override effective connection type" to 4G.
Labels: Needs-Triage-M68 Needs-Bisect
Labels: -Needs-Bisect
Looks like root cause is already identified in C#8, hence removing the needs-bisect label. Please feel free to add it back if this needs a bisect.

Thank you!
It's very curious. In our source code, the message level has been info-level over a year. On my Chromebook on 69.0.3497.35, I also confirm that it's still info-level message.

Adam, do you still see this error level message even on the latest 69 stable?

Also, "Group similar" function in the console worked for these messages, but it may not be able to group recently? I'm not familiar with how the console detect similar messages though.
Even without "group similar" disabled, the messages still showed up as errors. After updating to Chrome 69, I am not seeing the messages at all at the moment. But it could be they are not triggering because network speed is faster this morning. I will keep an eye on it throughout the day and report back.
Status: Fixed (was: Unconfirmed)
Looks like the root problem (i.e. the message is shown at the error level) is fixed and updating to Chrome 69 would have fixed it?  I'm tentatively closing this.

Adam- if you still see the problem after Chrome 69 feel free to reopen, or file a new one if the network detection needs to be improved further. Thanks for reporting & helpful feedback!

Sign in to add a comment