Bug Fix: Resize Event Trigger

Bug Fix: Resize Event Trigger

WebMediaPlayerMS triggers Resize event when the new sized frame
enters the player. However, one would need to wait some time
(~30 ms) for that frame to be displayed. Thus at the instant when
resize is triggered, javascript is still seeing the old size.

In this CL, we move the trigger to when |current_frame_| is updated.

BUG=604795
Committed: https://crrev.com/4089952731e1ad0a6cf83764042d59a41fbc42fe
Cr-Commit-Position: refs/heads/master@{#388635}

[qiangchen, 2016-04-19 17:40:07]

Description was changed from

==========
Bug Fix: Resize Event Trigger

BUG=
==========

to

==========
Bug Fix: Resize Event Trigger

BUG=https://buganizer.corp.google.com/u/0/issues/27029520
==========

[qiangchen, 2016-04-19 17:51:06]

Description was changed from

==========
Bug Fix: Resize Event Trigger

BUG=https://buganizer.corp.google.com/u/0/issues/27029520
==========

to

==========
Bug Fix: Resize Event Trigger

WebMediaPlayerMS triggers Resize event when the new sized frame
enters the player. However, one would need to wait some time
(~30 ms) for that frame to be displayed. Thus at the instant when
resize is triggered, javascript is still seeing the old size.

In this CL, we move the trigger to when |current_frame_| is updated.

BUG=604795
==========

[qiangchen, 2016-04-19 17:51:06]

qiangchen@chromium.org changed reviewers:
+ dalecurtis@chromium.org

[qiangchen, 2016-04-19 18:03:58]

This CL essentially fixes a bug visible on Hangouts.
We did not trigger resize event at correct instant.

PTAL.

[qiangchen, 2016-04-19 18:03:58]

This CL essentially fixes a bug visible on Hangouts.
We did not trigger resize event at correct instant.

PTAL.

[DaleCurtis, 2016-04-20 19:20:40]

Unittest? Otherwise looks good.

[qiangchen, 2016-04-20 22:16:42]

On 2016/04/20 19:20:40, DaleCurtis wrote:
> Unittest? Otherwise looks good.

Done.

[DaleCurtis, 2016-04-20 23:05:17]

lgtm

[qiangchen, 2016-04-20 23:33:34]

The CQ bit was checked by qiangchen@chromium.org

[commit-bot: I haz the power, 2016-04-20 23:37:45]

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

[commit-bot: I haz the power, 2016-04-21 00:29:37]

Description was changed from

==========
Bug Fix: Resize Event Trigger

WebMediaPlayerMS triggers Resize event when the new sized frame
enters the player. However, one would need to wait some time
(~30 ms) for that frame to be displayed. Thus at the instant when
resize is triggered, javascript is still seeing the old size.

In this CL, we move the trigger to when |current_frame_| is updated.

BUG=604795
==========

to

==========
Bug Fix: Resize Event Trigger

WebMediaPlayerMS triggers Resize event when the new sized frame
enters the player. However, one would need to wait some time
(~30 ms) for that frame to be displayed. Thus at the instant when
resize is triggered, javascript is still seeing the old size.

In this CL, we move the trigger to when |current_frame_| is updated.

BUG=604795
==========

[commit-bot: I haz the power, 2016-04-21 00:29:47]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-22 19:28:56]

Description was changed from

==========
Bug Fix: Resize Event Trigger

WebMediaPlayerMS triggers Resize event when the new sized frame
enters the player. However, one would need to wait some time
(~30 ms) for that frame to be displayed. Thus at the instant when
resize is triggered, javascript is still seeing the old size.

In this CL, we move the trigger to when |current_frame_| is updated.

BUG=604795
==========

to

==========
Bug Fix: Resize Event Trigger

WebMediaPlayerMS triggers Resize event when the new sized frame
enters the player. However, one would need to wait some time
(~30 ms) for that frame to be displayed. Thus at the instant when
resize is triggered, javascript is still seeing the old size.

In this CL, we move the trigger to when |current_frame_| is updated.

BUG=604795

Committed: https://crrev.com/4089952731e1ad0a6cf83764042d59a41fbc42fe
Cr-Commit-Position: refs/heads/master@{#388635}
==========

[commit-bot: I haz the power, 2016-04-22 19:28:59]

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