WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 161911
Remove non-standard postMessage overload
https://bugs.webkit.org/show_bug.cgi?id=161911
Summary
Remove non-standard postMessage overload
PhistucK
Reported
2016-09-13 09:34:54 PDT
https://github.com/WebKit/webkit/blob/da2308ab63592f69d45afcdccc7ee82a6a9edebe/Source/WebCore/bindings/js/JSDOMWindowCustom.cpp#L529
Chrome 54 removes it -
https://bugs.chromium.org/p/chromium/issues/detail?id=425896#c16
Attachments
Add attachment
proposed patch, testcase, etc.
Chris Dumez
Comment 1
2016-09-13 10:26:38 PDT
Looks like Safari is the only browser support the overload now. However, based on Chromium's intent to deprecate [1], usage is not that low (0.002%) and most usage is from Mac. [1]
https://groups.google.com/a/chromium.org/forum/#!msg/blink-dev/h4ooaB_Y9JE/nh7vXshGBwAJ
PhistucK
Comment 2
2016-09-13 12:59:18 PDT
The Blink threshold is 0.02%, so this is well below the threshold (I understand that you may have a different threshold). Rick, can you share the macOS only percentage?
PhistucK
Comment 3
2016-09-13 13:02:54 PDT
Also, it looks like the general metric is even lower now (0.0012).
https://www.chromestatus.com/metrics/feature/timeline/popularity/575
Sam Weinig
Comment 4
2016-12-05 16:14:05 PST
Fixed in
r209303
: <
http://trac.webkit.org/changeset/209303
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug