Rename WebEventHandler to MessageCallback

Rename WebEventHandler to MessageCallback

Rename WebEventHandler to MessageCallback to be more compatible with
other postmessage names.

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

[sgurun-gerrit only, 2015-03-18 21:47:45]

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

[sgurun-gerrit only, 2015-03-18 21:47:57]

On 2015/03/18 21:47:45, sgurun wrote:
> mailto:sgurun@chromium.org changed reviewers:
> + mailto:hush@chromium.org

PTAL, thanks

[hush (inactive), 2015-03-18 22:24:01]

lgtm with minor comments

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

https://codereview.chromium.org/1017333002/diff/1/android_webview/java/src/or...
android_webview/java/src/org/chromium/android_webview/MessagePort.java:75: * The
message callback for receiving messages. Called on main thread or if
could you say "call it on UI thread"?
main thread might be confused with the blink thread..
how about:
"Called on the UI thread or on the handler if provided"

[hush (inactive), 2015-03-18 22:24:28]

On 2015/03/18 22:24:01, hush wrote:
> lgtm with minor comments
> 
>
https://codereview.chromium.org/1017333002/diff/1/android_webview/java/src/or...
> File android_webview/java/src/org/chromium/android_webview/MessagePort.java
> (right):
> 
>
https://codereview.chromium.org/1017333002/diff/1/android_webview/java/src/or...
> android_webview/java/src/org/chromium/android_webview/MessagePort.java:75: *
The
> message callback for receiving messages. Called on main thread or if
> could you say "call it on UI thread"?
> main thread might be confused with the blink thread..
> how about:
> "Called on the UI thread or on the handler if provided"

I just edited your description so that bug # is the last line

[sgurun-gerrit only, 2015-03-18 22:37:54]

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

https://codereview.chromium.org/1017333002/diff/1/android_webview/java/src/or...
android_webview/java/src/org/chromium/android_webview/MessagePort.java:75: * The
message callback for receiving messages. Called on main thread or if
On 2015/03/18 22:24:01, hush wrote:
> could you say "call it on UI thread"?
> main thread might be confused with the blink thread..
> how about:
> "Called on the UI thread or on the handler if provided"

done. note that blink thread run at the renderer side

[sgurun-gerrit only, 2015-03-18 22:37:59]

The CQ bit was checked by sgurun@chromium.org

[sgurun-gerrit only, 2015-03-18 22:38:00]

The patchset sent to the CQ was uploaded after l-g-t-m from hush@chromium.org
Link to the patchset: https://codereview.chromium.org/1017333002/#ps20001
(title: "address nit")

[commit-bot: I haz the power, 2015-03-18 22:38:43]

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

[commit-bot: I haz the power, 2015-03-18 23:40:25]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-03-18 23:41:06]

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