Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HTML: self.reportError() #29738

Merged
merged 4 commits into from
Jul 27, 2021
Merged

HTML: self.reportError() #29738

merged 4 commits into from
Jul 27, 2021

Conversation

annevk
Copy link
Member

@annevk annevk commented Jul 21, 2021

No description provided.

Copy link
Member

@domenic domenic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we also test cross-origin muting? I know that's not very interoperable, but it might be worth seeing if a simple case is...

html/webappapis/scripting/reporterror.any.js Outdated Show resolved Hide resolved
html/webappapis/scripting/reporterror.any.js Show resolved Hide resolved
@annevk annevk requested a review from domenic July 23, 2021 10:20
@annevk
Copy link
Member Author

annevk commented Jul 23, 2021

I'd rather not touch cross-origin until we have more clarity on whatwg/html#958.

@annevk annevk merged commit c3cb110 into master Jul 27, 2021
@annevk annevk deleted the annevk/reporterror branch July 27, 2021 06:12
annevk added a commit to whatwg/html that referenced this pull request Jul 27, 2021
This allows libraries to emulate the behavior of an exception thrown in an event listener callback.

Additional context: whatwg/console#50.

Tests: web-platform-tests/wpt#29738.

Co-authored-by: Domenic Denicola <d@domenic.me>
webkit-commit-queue pushed a commit to WebKit/WebKit that referenced this pull request Aug 30, 2021
https://bugs.webkit.org/show_bug.cgi?id=228316
<rdar://problem/81446162>

Reviewed by Sam Weinig.

LayoutTests/imported/w3c:

Import test coverage from:
- web-platform-tests/wpt#29738

* web-platform-tests/html/webappapis/scripting/reporterror.any-expected.txt: Added.
* web-platform-tests/html/webappapis/scripting/reporterror.any.html: Added.
* web-platform-tests/html/webappapis/scripting/reporterror.any.js: Added.
(undefined.forEach.throwable.test.t.assert_equals):
(test):
* web-platform-tests/html/webappapis/scripting/reporterror.any.worker-expected.txt: Added.
* web-platform-tests/html/webappapis/scripting/reporterror.any.worker.html: Added.
* web-platform-tests/html/webappapis/scripting/w3c-import.log: Added.

Source/WebCore:

Implement self.reportError() as per:
- whatwg/html#1196

Firefox already shipped this and Chrome will do so soon too.

Tests: imported/w3c/web-platform-tests/html/webappapis/scripting/reporterror.any.html
       imported/w3c/web-platform-tests/html/webappapis/scripting/reporterror.any.worker.html

* page/DOMWindow.cpp:
(WebCore::DOMWindow::reportError):
* page/DOMWindow.h:
* page/WindowOrWorkerGlobalScope.idl:
* workers/WorkerGlobalScope.cpp:
(WebCore::WorkerGlobalScope::reportError):
* workers/WorkerGlobalScope.h:


Canonical link: https://commits.webkit.org/241098@main
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@281756 268f45cc-cd09-0410-ab3c-d52691b4dbfc
bertogg pushed a commit to Igalia/webkit that referenced this pull request Sep 6, 2021
https://bugs.webkit.org/show_bug.cgi?id=228316
<rdar://problem/81446162>

Reviewed by Sam Weinig.

LayoutTests/imported/w3c:

Import test coverage from:
- web-platform-tests/wpt#29738

* web-platform-tests/html/webappapis/scripting/reporterror.any-expected.txt: Added.
* web-platform-tests/html/webappapis/scripting/reporterror.any.html: Added.
* web-platform-tests/html/webappapis/scripting/reporterror.any.js: Added.
(undefined.forEach.throwable.test.t.assert_equals):
(test):
* web-platform-tests/html/webappapis/scripting/reporterror.any.worker-expected.txt: Added.
* web-platform-tests/html/webappapis/scripting/reporterror.any.worker.html: Added.
* web-platform-tests/html/webappapis/scripting/w3c-import.log: Added.

Source/WebCore:

Implement self.reportError() as per:
- whatwg/html#1196

Firefox already shipped this and Chrome will do so soon too.

Tests: imported/w3c/web-platform-tests/html/webappapis/scripting/reporterror.any.html
       imported/w3c/web-platform-tests/html/webappapis/scripting/reporterror.any.worker.html

* page/DOMWindow.cpp:
(WebCore::DOMWindow::reportError):
* page/DOMWindow.h:
* page/WindowOrWorkerGlobalScope.idl:
* workers/WorkerGlobalScope.cpp:
(WebCore::WorkerGlobalScope::reportError):
* workers/WorkerGlobalScope.h:


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@281756 268f45cc-cd09-0410-ab3c-d52691b4dbfc
mfreed7 pushed a commit to mfreed7/html that referenced this pull request Jun 3, 2022
This allows libraries to emulate the behavior of an exception thrown in an event listener callback.

Additional context: whatwg/console#50.

Tests: web-platform-tests/wpt#29738.

Co-authored-by: Domenic Denicola <d@domenic.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants