Null check content view core before using it.

Null check content view core before using it.

This is because content view core object could be destroyed at the time when
a method is called on it.

BUG=427684
Committed: https://crrev.com/854fc333d0a4b258ab3a64eaa4111fa45855ce9d
Cr-Commit-Position: refs/heads/master@{#301548}

[hush (inactive), 2014-10-27 22:17:12]

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

[hush (inactive), 2014-10-27 22:17:13]

PTAL

[hush (inactive), 2014-10-27 22:17:34]

On 2014/10/27 22:17:13, hush wrote:
> PTAL

internal bug b/18138575

[boliu, 2014-10-27 22:19:05]

boliu@chromium.org changed reviewers:
+ sievers@chromium.org

[boliu, 2014-10-27 22:19:06]

-> sievers for review for realz

https://codereview.chromium.org/677423003/diff/1/content/browser/renderer_hos...
File content/browser/renderer_host/render_widget_host_view_android.cc (right):

https://codereview.chromium.org/677423003/diff/1/content/browser/renderer_hos...
content/browser/renderer_host/render_widget_host_view_android.cc:1119: //
Content view core may possibly be destructed at this time.
No need for comment

https://codereview.chromium.org/677423003/diff/1/content/browser/renderer_hos...
content/browser/renderer_host/render_widget_host_view_android.cc:1120: if
(content_view_core_ == NULL)
if (!content_view_core_)

And I think you can put it at the top of the method

[hush (inactive), 2014-10-27 22:24:03]

https://codereview.chromium.org/677423003/diff/1/content/browser/renderer_hos...
File content/browser/renderer_host/render_widget_host_view_android.cc (right):

https://codereview.chromium.org/677423003/diff/1/content/browser/renderer_hos...
content/browser/renderer_host/render_widget_host_view_android.cc:1119: //
Content view core may possibly be destructed at this time.
On 2014/10/27 22:19:05, boliu wrote:
> No need for comment

Done.

https://codereview.chromium.org/677423003/diff/1/content/browser/renderer_hos...
content/browser/renderer_host/render_widget_host_view_android.cc:1120: if
(content_view_core_ == NULL)
On 2014/10/27 22:19:05, boliu wrote:
> if (!content_view_core_)
> 
> And I think you can put it at the top of the method

Done.

[no sievers, 2014-10-27 23:46:46]

lgtm

[hush (inactive), 2014-10-27 23:55:39]

The CQ bit was checked by hush@chromium.org

[commit-bot: I haz the power, 2014-10-28 00:02:05]

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

[commit-bot: I haz the power, 2014-10-28 02:15:35]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-10-28 02:16:05]

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