FrameView: Don't try to update the throttling status of a deleted frame

FrameView: Don't try to update the throttling status of a deleted frame

Because of the way the implicit |this| argument of a lambda and
WTF::bind() interact, we need to explicitly check if the given FrameView
was deleted before updating its throttling status.

BUG=673070
Committed: https://crrev.com/59e1644eb56dfa290da4ef5a6173a00484404904
Cr-Commit-Position: refs/heads/master@{#440161}

[Sami, 2016-12-21 15:56:14]

The CQ bit was checked by skyostil@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-21 15:56:49]

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

[Sami, 2016-12-21 15:57:01]

Description was changed from

==========
FrameView: Don't try to update the throttling status of a deleted frame

Because of the way the implicit |this| argument of a lambda and
WTF::bind() interact, we need to explicitly check if the given FrameView
was deleted before updating its throttling status.

BUG=673070
==========

to

==========
FrameView: Don't try to update the throttling status of a deleted frame

Because of the way the implicit |this| argument of a lambda and
WTF::bind() interact, we need to explicitly check if the given FrameView
was deleted before updating its throttling status.

BUG=673070
==========

[Sami, 2016-12-21 15:57:01]

skyostil@chromium.org changed reviewers:
+ dcheng@chromium.org

[dcheng, 2016-12-21 17:22:11]

LGTM

[commit-bot: I haz the power, 2016-12-21 17:22:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-21 17:22:23]

Dry run: This issue passed the CQ dry run.

[Sami, 2016-12-21 18:27:31]

The CQ bit was checked by skyostil@chromium.org

[commit-bot: I haz the power, 2016-12-21 18:28:09]

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-12-21 18:35:17]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482344851982330, "parent_rev":
"b9742cf37427208a54c4af4d941fd87ace47c0b8", "commit_rev":
"f1d6b3793e4601289b35bd2518ae4d7addceb95e"}

[commit-bot: I haz the power, 2016-12-21 18:35:46]

Description was changed from

==========
FrameView: Don't try to update the throttling status of a deleted frame

Because of the way the implicit |this| argument of a lambda and
WTF::bind() interact, we need to explicitly check if the given FrameView
was deleted before updating its throttling status.

BUG=673070
==========

to

==========
FrameView: Don't try to update the throttling status of a deleted frame

Because of the way the implicit |this| argument of a lambda and
WTF::bind() interact, we need to explicitly check if the given FrameView
was deleted before updating its throttling status.

BUG=673070

Review-Url: https://codereview.chromium.org/2591243002
==========

[commit-bot: I haz the power, 2016-12-21 18:35:47]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-21 18:37:50]

Description was changed from

==========
FrameView: Don't try to update the throttling status of a deleted frame

Because of the way the implicit |this| argument of a lambda and
WTF::bind() interact, we need to explicitly check if the given FrameView
was deleted before updating its throttling status.

BUG=673070

Review-Url: https://codereview.chromium.org/2591243002
==========

to

==========
FrameView: Don't try to update the throttling status of a deleted frame

Because of the way the implicit |this| argument of a lambda and
WTF::bind() interact, we need to explicitly check if the given FrameView
was deleted before updating its throttling status.

BUG=673070

Committed: https://crrev.com/59e1644eb56dfa290da4ef5a6173a00484404904
Cr-Commit-Position: refs/heads/master@{#440161}
==========

[commit-bot: I haz the power, 2016-12-21 18:37:51]

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