Rename some postmessage API to make it more meaningful.

Rename some postmessage API to make it more meaningful.

BUG=393291
Committed: https://crrev.com/f9dbeb215e44d6f46e8fa395d1a6bb114a9752bf
Cr-Commit-Position: refs/heads/master@{#316980}

[sgurun-gerrit only, 2015-02-19 01:48:27]

sgurun@chromium.org changed reviewers:
+ hush@chromium.org

[sgurun-gerrit only, 2015-02-19 01:48:34]

On 2015/02/19 01:48:27, sgurun wrote:
> mailto:sgurun@chromium.org changed reviewers:
> + mailto:hush@chromium.org

PTAL, thanks

[hush (inactive), 2015-02-19 02:05:24]

https://codereview.chromium.org/929393003/diff/1/android_webview/java/src/org...
File android_webview/java/src/org/chromium/android_webview/MessagePort.java
(right):

https://codereview.chromium.org/929393003/diff/1/android_webview/java/src/org...
android_webview/java/src/org/chromium/android_webview/MessagePort.java:43:
public abstract static class WebEventHandler {
WebEvent implies there is something more than just post message related?
Maybe just WebMessageHandler?

https://codereview.chromium.org/929393003/diff/1/android_webview/java/src/org...
android_webview/java/src/org/chromium/android_webview/MessagePort.java:116:
public void setWebEventHandler(WebEventHandler webEventHandler, Handler handler)
{
remove this function??

[sgurun-gerrit only, 2015-02-19 02:30:36]

https://codereview.chromium.org/929393003/diff/1/android_webview/java/src/org...
File android_webview/java/src/org/chromium/android_webview/MessagePort.java
(right):

https://codereview.chromium.org/929393003/diff/1/android_webview/java/src/org...
android_webview/java/src/org/chromium/android_webview/MessagePort.java:43:
public abstract static class WebEventHandler {
On 2015/02/19 02:05:24, hush wrote:
> WebEvent implies there is something more than just post message related?
> Maybe just WebMessageHandler?

it is a good suggestion but HTML5 spec calls it an EventHandler, in MessagePort.
Let's keep it like this, and let's reconsider. I will put a note in design doc.

https://codereview.chromium.org/929393003/diff/1/android_webview/java/src/org...
android_webview/java/src/org/chromium/android_webview/MessagePort.java:116:
public void setWebEventHandler(WebEventHandler webEventHandler, Handler handler)
{
On 2015/02/19 02:05:24, hush wrote:
> remove this function??

Done.

[hush (inactive), 2015-02-19 02:31:14]

The CQ bit was checked by hush@chromium.org

[hush (inactive), 2015-02-19 02:31:15]

lgtm

[commit-bot: I haz the power, 2015-02-19 02:32:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/929393003/20001

[commit-bot: I haz the power, 2015-02-19 02:52:15]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-02-19 02:52:59]

Patchset 2 (id:??) landed as
https://crrev.com/f9dbeb215e44d6f46e8fa395d1a6bb114a9752bf
Cr-Commit-Position: refs/heads/master@{#316980}