Add a utility method to receive an AwContents

Add a utility method to receive an AwContents from a render_process_id and a render_host_id. We need to do this conversion in multiple (at least two) places so having the utility method helps.

BUG=


Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=181325

[sgurun-gerrit only, 2013-02-07 03:18:37]

please take a look, thanks.

[Kristian Monsen, 2013-02-07 03:30:55]

One comment, otherwise looks good to me. Thanks for adding.

https://codereview.chromium.org/12207061/diff/2001/android_webview/native/aw_...
File android_webview/native/aw_contents.cc (right):

https://codereview.chromium.org/12207061/diff/2001/android_webview/native/aw_...
android_webview/native/aw_contents.cc:121: const content::RenderViewHost* rvh =
Should this only be used on the UI thread? In that case it should be DCHECK'ed.

[sgurun-gerrit only, 2013-02-07 03:37:32]

https://codereview.chromium.org/12207061/diff/2001/android_webview/native/aw_...
File android_webview/native/aw_contents.cc (right):

https://codereview.chromium.org/12207061/diff/2001/android_webview/native/aw_...
android_webview/native/aw_contents.cc:121: const content::RenderViewHost* rvh =
One of the methods that the FromID calls already checks for it but let me put
another one here from better visibility.

On 2013/02/07 03:30:55, Kristian Monsen wrote:
> Should this only be used on the UI thread? In that case it should be
DCHECK'ed.

[sgurun-gerrit only, 2013-02-07 03:39:08]

added a dcheck. PTAL,

[Kristian Monsen, 2013-02-07 03:46:44]

Thanks I think that is clearer. Now it should be obvious to anyone calling this
that it can only be used on the UI thread. LGTM (but I am not a contributor or
an owner.)

[mkosiba (inactive), 2013-02-07 09:44:11]

lgtm

[benm (inactive), 2013-02-07 10:23:06]

lgtm

[commit-bot: I haz the power, 2013-02-07 15:09:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sgurun@chromium.org/12207061/9001

[commit-bot: I haz the power, 2013-02-07 17:05:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sgurun@chromium.org/12207061/9001

[commit-bot: I haz the power, 2013-02-07 17:14:50]

Change committed as 181325