Move PostMessageToFrame to WebContentsAndroid

Move PostMessageToFrame to WebContentsAndroid

Move PostMessageToFrame method from ContentViewCore to WebContentsAndroid
which is a more suitable location.

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

[sgurun-gerrit only, 2014-10-21 18:01:53]

Patchset #2 (id:20001) has been deleted

[sgurun-gerrit only, 2014-10-21 18:28:57]

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

[sgurun-gerrit only, 2014-10-21 18:30:05]

On 2014/10/21 18:28:57, sgurun wrote:
> mailto:sgurun@chromium.org changed reviewers:
> + mailto:tedchoc@chromium.org

PTAL, thanks

[Ted C, 2014-10-21 21:36:57]

https://codereview.chromium.org/649933006/diff/40001/content/browser/web_cont...
File content/browser/web_contents/web_contents_android.cc (right):

https://codereview.chromium.org/649933006/diff/40001/content/browser/web_cont...
content/browser/web_contents/web_contents_android.cc:375: return;
indented two too many

https://codereview.chromium.org/649933006/diff/40001/content/public/android/j...
File
content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
(right):

https://codereview.chromium.org/649933006/diff/40001/content/public/android/j...
content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java:1182:
public void postMessageToFrame(String frameName, String message,
I don't see any callers to this function outside of a test.  It should be
removed and people should call web contents directly.

[sgurun-gerrit only, 2014-10-21 23:43:35]

https://codereview.chromium.org/649933006/diff/40001/content/browser/web_cont...
File content/browser/web_contents/web_contents_android.cc (right):

https://codereview.chromium.org/649933006/diff/40001/content/browser/web_cont...
content/browser/web_contents/web_contents_android.cc:375: return;
On 2014/10/21 21:36:56, Ted C wrote:
> indented two too many

Done.

https://codereview.chromium.org/649933006/diff/40001/content/public/android/j...
File
content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
(right):

https://codereview.chromium.org/649933006/diff/40001/content/public/android/j...
content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java:1182:
public void postMessageToFrame(String frameName, String message,
On 2014/10/21 21:36:57, Ted C wrote:
> I don't see any callers to this function outside of a test.  It should be
> removed and people should call web contents directly.

Done.

[Ted C, 2014-10-21 23:44:52]

lgtm

[sgurun-gerrit only, 2014-10-21 23:47:17]

The CQ bit was checked by sgurun@chromium.org

[commit-bot: I haz the power, 2014-10-21 23:50:40]

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

[commit-bot: I haz the power, 2014-10-22 01:56:35]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2014-10-22 04:05:18]

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