Bug 161911
| Summary: | Remove non-standard postMessage overload | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | PhistucK <phistuck> |
| Component: | Bindings | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | ap, cdumez, rbyers, sam |
| Priority: | P2 | ||
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
PhistucK
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
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
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
Also, it looks like the general metric is even lower now (0.0012). https://www.chromestatus.com/metrics/feature/timeline/popularity/575
Sam Weinig
Fixed in r209303: <http://trac.webkit.org/changeset/209303>