Deprecate window.postMessage(m, transferables, origin) overload.

Deprecate window.postMessage(m, transferables, origin) overload.

This argument-swapped legacy overload has little use, hence it
is time to try to phase it out. Deprecate with view to removing
in M54.

Intent: https://groups.google.com/a/chromium.org/d/msg/blink-dev/h4ooaB_Y9JE/nh7vXshGBwAJ

R=
BUG=425896
Committed: https://crrev.com/48b609eca30c2bbcb91d74bc311f0fd34b030ab4
Cr-Commit-Position: refs/heads/master@{#389800}

[sof, 2016-04-21 08:37:12]

Description was changed from

==========
Deprecate window.PostMessage(m, transferables, origin) overload.

This argument-swapped legacy overload has little use, hence it
is time to try to phase it out. Deprecate with view to removing
in M54.

R=
BUG=425896
==========

to

==========
Deprecate window.postMessage(m, transferables, origin) overload.

This argument-swapped legacy overload has little use, hence it
is time to try to phase it out. Deprecate with view to removing
in M54.

R=
BUG=425896
==========

[sof, 2016-04-21 09:51:52]

sigbjornf@opera.com changed reviewers:
+ jochen@chromium.org, philipj@opera.com

[sof, 2016-04-21 09:51:53]

please take a look.

Following up on


https://groups.google.com/a/chromium.org/d/msg/blink-dev/h4ooaB_Y9JE/nh7vXshG...

[jochen (gone - plz use gerrit), 2016-04-21 09:59:13]

lgtm

[philipj_slow, 2016-04-21 10:03:29]

lgtm

https://codereview.chromium.org/1903873004/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/frame/Deprecation.cpp (right):

https://codereview.chromium.org/1903873004/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/frame/Deprecation.cpp:368: return
willBeRemoved("'window.postMessage(message, transferables, targetOrigin)'", 54,
"5719033043222528");
Maybe replacedWillBeRemoved so that people don't have to spend any time figuring
out what to do instead?

[sof, 2016-04-21 10:18:39]

https://codereview.chromium.org/1903873004/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/frame/Deprecation.cpp (right):

https://codereview.chromium.org/1903873004/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/frame/Deprecation.cpp:368: return
willBeRemoved("'window.postMessage(message, transferables, targetOrigin)'", 54,
"5719033043222528");
On 2016/04/21 10:03:29, philipj_slow wrote:
> Maybe replacedWillBeRemoved so that people don't have to spend any time
figuring
> out what to do instead?

Good idea, this seems like a poster child for that helper method :) Done.

[sof, 2016-04-21 10:21:50]

Description was changed from

==========
Deprecate window.postMessage(m, transferables, origin) overload.

This argument-swapped legacy overload has little use, hence it
is time to try to phase it out. Deprecate with view to removing
in M54.

R=
BUG=425896
==========

to

==========
Deprecate window.postMessage(m, transferables, origin) overload.

This argument-swapped legacy overload has little use, hence it
is time to try to phase it out. Deprecate with view to removing
in M54.

Intent:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/h4ooaB_Y9JE/nh7vXshG...

R=
BUG=425896
==========

[haraken, 2016-04-21 10:26:12]

LGTM

[sof, 2016-04-26 14:53:59]

moving ahead w/ deprecation-to-be-removed-in-M54.

[sof, 2016-04-26 14:54:05]

The CQ bit was checked by sigbjornf@opera.com

[sof, 2016-04-26 14:54:06]

The patchset sent to the CQ was uploaded after l-g-t-m from philipj@opera.com,
jochen@chromium.org, haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/1903873004/#ps80001
(title: "rebased")

[commit-bot: I haz the power, 2016-04-26 14:54:33]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1903873004/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1903873004/80001

[commit-bot: I haz the power, 2016-04-26 16:19:53]

Description was changed from

==========
Deprecate window.postMessage(m, transferables, origin) overload.

This argument-swapped legacy overload has little use, hence it
is time to try to phase it out. Deprecate with view to removing
in M54.

Intent:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/h4ooaB_Y9JE/nh7vXshG...

R=
BUG=425896
==========

to

==========
Deprecate window.postMessage(m, transferables, origin) overload.

This argument-swapped legacy overload has little use, hence it
is time to try to phase it out. Deprecate with view to removing
in M54.

Intent:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/h4ooaB_Y9JE/nh7vXshG...

R=
BUG=425896
==========

[commit-bot: I haz the power, 2016-04-26 16:19:54]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-04-26 16:21:30]

Description was changed from

==========
Deprecate window.postMessage(m, transferables, origin) overload.

This argument-swapped legacy overload has little use, hence it
is time to try to phase it out. Deprecate with view to removing
in M54.

Intent:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/h4ooaB_Y9JE/nh7vXshG...

R=
BUG=425896
==========

to

==========
Deprecate window.postMessage(m, transferables, origin) overload.

This argument-swapped legacy overload has little use, hence it
is time to try to phase it out. Deprecate with view to removing
in M54.

Intent:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/h4ooaB_Y9JE/nh7vXshG...

R=
BUG=425896

Committed: https://crrev.com/48b609eca30c2bbcb91d74bc311f0fd34b030ab4
Cr-Commit-Position: refs/heads/master@{#389800}
==========

[commit-bot: I haz the power, 2016-04-26 16:21:31]

Patchset 5 (id:??) landed as
https://crrev.com/48b609eca30c2bbcb91d74bc311f0fd34b030ab4
Cr-Commit-Position: refs/heads/master@{#389800}