1927801 - [wpt-sync] Sync PR 48853 - Display resize observer loop notifications in devtools

Status: RESOLVED FIXED

(Core :: Layout, task, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Sync web-platform-tests PR 48853 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/48853
Details from upstream follow.

Philip Rogers <[e-mail]> wrote:

Display resize observer loop notifications in devtools

Resize observer loop notifications errors that are not canceled should
appear in the devtools console so authors have some indication of the
error. This patch does this with two changes:

1. A valid script state is used so the exception is not dropped in
   MainThreadDebugger::ExceptionThrown.
2. A ScriptValue of the error message is used so a message appears in
   the console. Without a ScriptValue, MainThreadDebugger::ExceptionThrown
   will print "Uncaught null".

The console behavior is not specified so two tentative tests have been
added (Safari passes these). A TODO has been added to provide a useful
source location, as suggested by https://crbug.com/40667137#comment10.

Fixed: 40667137
Change-Id: Id20c977efd1fc03caba517ecec3bf3b0a9f53163
Reviewed-on: https://chromium-review.googlesource.com/5970814
WPT-Export-Revision: 10c6f545456fc786380c4462696cedb9313a4f1d

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=8c004ab875061055e80266c103e52d4c79161b3e

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6f86f5d845c4
[wpt PR 48853] - Display resize observer loop notifications in devtools, a=testonly

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Test result changes from PR not available.

Comment 4

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/6f86f5d845c4