Early out of registerViewportLayerWithCompositor if there's no local MainFrame.

Early out of registerViewportLayerWithCompositor if there's no local MainFrame.

I moved this code out of VisualViewport in r404359 but forgot this one check
and early-out. The crashes started happening right after that so it's likely the
culprit.

BUG=627442
Committed: https://crrev.com/1af81f87b72e4ed57341800a33153beae03f5f3f
Cr-Commit-Position: refs/heads/master@{#405168}

[bokan, 2016-07-12 18:13:11]

Description was changed from

==========
Early out of registerViewportLayerWithCompositor if there's no local MainFrame.

I moved this code out of VisualViewport in r404359 but forgot this one check
and early-out. The crashes started happening right after that so it's likely the
culprit.

BUG=627442
==========

to

==========
Early out of registerViewportLayerWithCompositor if there's no local MainFrame.

I moved this code out of VisualViewport in r404359 but forgot this one check
and early-out. The crashes started happening right after that so it's likely the
culprit.

BUG=627442
==========

[bokan, 2016-07-12 18:13:12]

bokan@chromium.org changed reviewers:
+ dtapuska@chromium.org

[bokan, 2016-07-12 18:15:09]

Dave, this was very likely caused by https://codereview.chromium.org/2128363002/
which should have been a straight move but I forgot that one check. The
callstack seems a little weird if the frame were NULL but maybe there's some
optimization rearranging things...

https://codereview.chromium.org/2113483002/ changes this method around some and
is about to land so I rebased over it but I think the problem would otherwise
still be there just with a new stacktrace.

[dtapuska, 2016-07-12 18:25:21]

On 2016/07/12 18:15:09, bokan wrote:
> Dave, this was very likely caused by
https://codereview.chromium.org/2128363002/
> which should have been a straight move but I forgot that one check. The
> callstack seems a little weird if the frame were NULL but maybe there's some
> optimization rearranging things...
> 
> https://codereview.chromium.org/2113483002/ changes this method around some
and
> is about to land so I rebased over it but I think the problem would otherwise
> still be there just with a new stacktrace.

lgtm

[bokan, 2016-07-12 23:03:52]

The CQ bit was checked by bokan@chromium.org

[bokan, 2016-07-12 23:03:57]

The CQ bit was unchecked by bokan@chromium.org

[bokan, 2016-07-13 14:49:57]

The CQ bit was checked by bokan@chromium.org

[commit-bot: I haz the power, 2016-07-13 14:50:07]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-13 15:56:34]

Description was changed from

==========
Early out of registerViewportLayerWithCompositor if there's no local MainFrame.

I moved this code out of VisualViewport in r404359 but forgot this one check
and early-out. The crashes started happening right after that so it's likely the
culprit.

BUG=627442
==========

to

==========
Early out of registerViewportLayerWithCompositor if there's no local MainFrame.

I moved this code out of VisualViewport in r404359 but forgot this one check
and early-out. The crashes started happening right after that so it's likely the
culprit.

BUG=627442
==========

[commit-bot: I haz the power, 2016-07-13 15:56:35]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-13 15:56:54]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-13 15:59:07]

Description was changed from

==========
Early out of registerViewportLayerWithCompositor if there's no local MainFrame.

I moved this code out of VisualViewport in r404359 but forgot this one check
and early-out. The crashes started happening right after that so it's likely the
culprit.

BUG=627442
==========

to

==========
Early out of registerViewportLayerWithCompositor if there's no local MainFrame.

I moved this code out of VisualViewport in r404359 but forgot this one check
and early-out. The crashes started happening right after that so it's likely the
culprit.

BUG=627442

Committed: https://crrev.com/1af81f87b72e4ed57341800a33153beae03f5f3f
Cr-Commit-Position: refs/heads/master@{#405168}
==========

[commit-bot: I haz the power, 2016-07-13 15:59:09]

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