[m48 merge] Remove clientWidth call during MediaControls::reset.

Remove clientWidth call during MediaControls::reset.

This is a merge to M48 of f003b7d5f0f8667873a8f152d5a06512d3bded9c .

Avoid calling clientWidth early to get the control panel width.
It can cause a crash when the HitTestCache is cleared, if the
cache was the last holder to the media element.  Normally, this
can't happen, but a callback from the WebMediaPlayer can be
scheduled between the cache becoming the last reference and the
next layout.

BUG=531259
NOTRY=true
NOPRESUBMIT=true

[liberato (no reviews please), 2016-01-11 19:09:58]

Description was changed from

==========
Remove clientWidth call during MediaControls::reset.

This is a merge to M48 of f003b7d5f0f8667873a8f152d5a06512d3bded9c .

Avoid calling clientWidth early to get the control panel width.
It can cause a crash when the HitTestCache is cleared, if the
cache was the last holder to the media element.  Normally, this
can't happen, but a callback from the WebMediaPlayer can be
scheduled between the cache becoming the last reference and the
next layout.

BUG=531259
==========

to

==========
Remove clientWidth call during MediaControls::reset.

This is a merge to M48 of f003b7d5f0f8667873a8f152d5a06512d3bded9c .

Avoid calling clientWidth early to get the control panel width.
It can cause a crash when the HitTestCache is cleared, if the
cache was the last holder to the media element.  Normally, this
can't happen, but a callback from the WebMediaPlayer can be
scheduled between the cache becoming the last reference and the
next layout.

BUG=531259
==========

[liberato (no reviews please), 2016-01-11 19:09:58]

liberato@chromium.org changed reviewers:
+ philipj@opera.com

[philipj_slow, 2016-01-12 15:15:26]

lgtm

[philipj_slow, 2016-01-12 15:15:45]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2016-01-12 15:16:02]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1578853002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1578853002/1

[commit-bot: I haz the power, 2016-01-12 15:16:03]

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

[commit-bot: I haz the power, 2016-01-12 15:16:03]

CLs for remote refs other than refs/pending/heads/master must contain NOTRY=true
and NOPRESUBMIT=true in order for the CQ to process them

[liberato (no reviews please), 2016-01-12 17:49:11]

The CQ bit was checked by liberato@chromium.org

[liberato (no reviews please), 2016-01-12 17:49:11]

The patchset sent to the CQ was uploaded after l-g-t-m from philipj@opera.com
Link to the patchset: https://codereview.chromium.org/1578853002/#ps20001
(title: "nopresubmit / notry")

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

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

[commit-bot: I haz the power, 2016-01-12 17:50:23]

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

[commit-bot: I haz the power, 2016-01-12 17:50:23]

CLs for remote refs other than refs/pending/heads/master must contain NOTRY=true
and NOPRESUBMIT=true in order for the CQ to process them

[liberato (no reviews please), 2016-01-12 17:51:19]

Description was changed from

==========
Remove clientWidth call during MediaControls::reset.

This is a merge to M48 of f003b7d5f0f8667873a8f152d5a06512d3bded9c .

Avoid calling clientWidth early to get the control panel width.
It can cause a crash when the HitTestCache is cleared, if the
cache was the last holder to the media element.  Normally, this
can't happen, but a callback from the WebMediaPlayer can be
scheduled between the cache becoming the last reference and the
next layout.

BUG=531259
==========

to

==========
Remove clientWidth call during MediaControls::reset.

This is a merge to M48 of f003b7d5f0f8667873a8f152d5a06512d3bded9c .

Avoid calling clientWidth early to get the control panel width.
It can cause a crash when the HitTestCache is cleared, if the
cache was the last holder to the media element.  Normally, this
can't happen, but a callback from the WebMediaPlayer can be
scheduled between the cache becoming the last reference and the
next layout.

BUG=531259
NOTRY=true
NOPRESUBMIT=true
==========

[liberato (no reviews please), 2016-01-12 17:51:30]

The CQ bit was checked by liberato@chromium.org

[commit-bot: I haz the power, 2016-01-12 17:51:37]

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

[commit-bot: I haz the power, 2016-01-12 17:53:39]

Description was changed from

==========
Remove clientWidth call during MediaControls::reset.

This is a merge to M48 of f003b7d5f0f8667873a8f152d5a06512d3bded9c .

Avoid calling clientWidth early to get the control panel width.
It can cause a crash when the HitTestCache is cleared, if the
cache was the last holder to the media element.  Normally, this
can't happen, but a callback from the WebMediaPlayer can be
scheduled between the cache becoming the last reference and the
next layout.

BUG=531259
NOTRY=true
NOPRESUBMIT=true
==========

to

==========
Remove clientWidth call during MediaControls::reset.

This is a merge to M48 of f003b7d5f0f8667873a8f152d5a06512d3bded9c .

Avoid calling clientWidth early to get the control panel width.
It can cause a crash when the HitTestCache is cleared, if the
cache was the last holder to the media element.  Normally, this
can't happen, but a callback from the WebMediaPlayer can be
scheduled between the cache becoming the last reference and the
next layout.

BUG=531259
NOTRY=true
NOPRESUBMIT=true
==========

[commit-bot: I haz the power, 2016-01-12 17:53:39]

Committed patchset #2 (id:20001)