161911 – Remove non-standard postMessage overload

Bug 161911 - Remove non-standard postMessage overload

Summary: Remove non-standard postMessage overload

Status:	RESOLVED FIXED

Alias:	None

Product:	WebKit
Classification:	Unclassified
Component:	Bindings (show other bugs)
Version:	WebKit Nightly Build
Hardware:	Unspecified Unspecified

Importance:	P2 Normal
Assignee:	Nobody

URL:
Keywords:

Depends on:
Blocks:

Reported:	2016-09-13 09:34 PDT by PhistucK
Modified:	2016-12-05 16:14 PST (History)
CC List:	4 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description PhistucK 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

Comment 1 Chris Dumez 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

Comment 2 PhistucK 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?

Comment 3 PhistucK 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

Comment 4 Sam Weinig 2016-12-05 16:14:05 PST

Fixed in r209303: <http://trac.webkit.org/changeset/209303>