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
[Deprecation Reporting] Stub implementation #4409
[Deprecation Reporting] Stub implementation #4409
Conversation
EWS run on previous version of this PR (hash 8dbf8ee) |
8dbf8ee
to
f1f055e
Compare
EWS run on previous version of this PR (hash f1f055e) |
f1f055e
to
08d3884
Compare
EWS run on previous version of this PR (hash 08d3884) |
08d3884
to
0f13c35
Compare
EWS run on current version of this PR (hash 0f13c35) |
|
||
const AtomString& DeprecationReportBody::deprecationReportType() | ||
{ | ||
static NeverDestroyed<AtomString> reportType { "deprecation"_s }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this is exposed to both main thread and workers, we likely want something like:
static NeverDestroyed<const AtomString> reportType(MAKE_STATIC_STRING_IMPL("deprecation"));
https://bugs.webkit.org/show_bug.cgi?id=245245 <rdar://problem/99988574> Reviewed by Chris Dumez. Import the Web Platform Tests for Deprecation Reporting, create a new Experimental Feature to represent the work, and provide a stub implementation of the report * LayoutTests/imported/w3c/resources/import-expectations.json: * LayoutTests/imported/w3c/web-platform-tests/deprecation-reporting/META.yml: Added. * LayoutTests/imported/w3c/web-platform-tests/deprecation-reporting/idlharness.any-expected.txt: Added. * LayoutTests/imported/w3c/web-platform-tests/deprecation-reporting/idlharness.any.html: Added. * LayoutTests/imported/w3c/web-platform-tests/deprecation-reporting/idlharness.any.js: Added. * LayoutTests/imported/w3c/web-platform-tests/deprecation-reporting/idlharness.any.worker-expected.txt: Added. * LayoutTests/imported/w3c/web-platform-tests/deprecation-reporting/idlharness.any.worker.html: Added. * LayoutTests/imported/w3c/web-platform-tests/deprecation-reporting/w3c-import.log: Added. * Source/WTF/Scripts/Preferences/WebPreferencesExperimental.yaml: * Source/WebCore/CMakeLists.txt: * Source/WebCore/DerivedSources-input.xcfilelist: * Source/WebCore/DerivedSources-output.xcfilelist: * Source/WebCore/DerivedSources.make: * Source/WebCore/Headers.cmake: * Source/WebCore/Modules/reporting/DeprecationReportBody.cpp: Added. (WebCore::DeprecationReportBody::deprecationReportType): (WebCore::DeprecationReportBody::DeprecationReportBody): (WebCore::DeprecationReportBody::create): (WebCore::DeprecationReportBody::type const): (WebCore::DeprecationReportBody::id const): (WebCore::DeprecationReportBody::anticipatedRemoval const): (WebCore::DeprecationReportBody::message const): (WebCore::DeprecationReportBody::sourceFile const): (WebCore::DeprecationReportBody::lineNumber const): (WebCore::DeprecationReportBody::columnNumber const): (WebCore::DeprecationReportBody::createReportFormDataForViolation const): * Source/WebCore/Modules/reporting/DeprecationReportBody.h: Copied from Source/WebCore/Modules/reporting/TestReportBody.h. (WebCore::DeprecationReportBody::encode const): (WebCore::DeprecationReportBody::decode): (isType): * Source/WebCore/Modules/reporting/DeprecationReportBody.idl: Copied from Source/WebCore/Modules/reporting/ViolationReportType.h. * Source/WebCore/Modules/reporting/ReportingObserver.cpp: (WebCore::isVisibleToReportingObservers): * Source/WebCore/Modules/reporting/TestReportBody.h: * Source/WebCore/Modules/reporting/ViolationReportType.h: * Source/WebCore/Sources.txt: * Source/WebCore/WebCore.xcodeproj/project.pbxproj: * Source/WebCore/bindings/js/JSReportBodyCustom.cpp: (WebCore::toJSNewlyCreated): * Source/WebCore/bindings/js/WebCoreBuiltinNames.h: * Source/WebCore/loader/PingLoader.cpp: (WebCore::PingLoader::sendViolationReport): * Source/WebKit/Shared/WebCoreArgumentCoders.cpp: (IPC::ArgumentCoder<RefPtr<WebCore::ReportBody>>::encode): (IPC::ArgumentCoder<RefPtr<WebCore::ReportBody>>::decode): Canonical link: https://commits.webkit.org/254634@main
0f13c35
to
a01206b
Compare
Committed 254634@main (a01206b): https://commits.webkit.org/254634@main Reviewed commits have been landed. Closing PR #4409 and removing active labels. |
a01206b
0f13c35