Fix a crash when setting MediaControlsEnabled setting for videos without controls

Fix a crash when setting MediaControlsEnabled setting for videos without controls

There's a crash which is triggered when we set MediaControlsEnabled
setting before the video controls get created. This CL adds null-check
before calling MediaControls::onMediaControlsEnabledChange().

BUG=696645
Review-Url: https://codereview.chromium.org/2722623002
Cr-Commit-Position: refs/heads/master@{#453595}
Committed: https://chromium.googlesource.com/chromium/src/+/e48ec244742e350bb55ad07dde1150022c536f84

[Zhiqiang Zhang (Slow), 2017-02-27 18:42:35]

zqzhang@chromium.org changed reviewers:
+ mlamouri@chromium.org

[Zhiqiang Zhang (Slow), 2017-02-27 18:42:43]

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

[commit-bot: I haz the power, 2017-02-27 18:43:21]

Dry run: 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, 2017-02-27 22:28:31]

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

[commit-bot: I haz the power, 2017-02-27 22:28:32]

Dry run: This issue passed the CQ dry run.

[mlamouri (slow - plz ping), 2017-02-28 14:30:11]

The CQ bit was checked by mlamouri@chromium.org

[mlamouri (slow - plz ping), 2017-02-28 14:30:12]

lgtm

[commit-bot: I haz the power, 2017-02-28 14:30:25]

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, 2017-02-28 14:34:02]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488292211802510, "parent_rev":
"6bfad9134c3101d8f43a9b9086f7c96b1d08e4af", "commit_rev":
"e48ec244742e350bb55ad07dde1150022c536f84"}

[commit-bot: I haz the power, 2017-02-28 14:34:38]

Description was changed from

==========
Fix a crash when setting MediaControlsEnabled setting for videos without
controls

There's a crash which is triggered when we set MediaControlsEnabled
setting before the video controls get created. This CL adds null-check
before calling MediaControls::onMediaControlsEnabledChange().

BUG=696645
==========

to

==========
Fix a crash when setting MediaControlsEnabled setting for videos without
controls

There's a crash which is triggered when we set MediaControlsEnabled
setting before the video controls get created. This CL adds null-check
before calling MediaControls::onMediaControlsEnabledChange().

BUG=696645

Review-Url: https://codereview.chromium.org/2722623002
Cr-Commit-Position: refs/heads/master@{#453595}
Committed:
https://chromium.googlesource.com/chromium/src/+/e48ec244742e350bb55ad07dde11...
==========

[commit-bot: I haz the power, 2017-02-28 14:34:39]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/e48ec244742e350bb55ad07dde11...