Bug Fix: Resize Event Trigger (M51)

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
Review URL: https://codereview.chromium.org/1895233003

Cr-Commit-Position: refs/heads/master@{#388635}
(cherry picked from commit 4089952731e1ad0a6cf83764042d59a41fbc42fe)

NOTRY=true
NOPRESUBMIT=true

[qiangchen, 2016-04-25 17:11:26]

Patchset #1 (id:1) has been deleted

[qiangchen, 2016-04-25 17:11:44]

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

Review URL: https://codereview.chromium.org/1895233003

Cr-Commit-Position: refs/heads/master@{#388635}
(cherry picked from commit 4089952731e1ad0a6cf83764042d59a41fbc42fe)
==========

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

Review URL: https://codereview.chromium.org/1895233003

Cr-Commit-Position: refs/heads/master@{#388635}
(cherry picked from commit 4089952731e1ad0a6cf83764042d59a41fbc42fe)
==========

[qiangchen, 2016-04-25 17:11:44]

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

[qiangchen, 2016-04-25 17:13:39]

The merge CL.

Can you approve it?

[DaleCurtis, 2016-04-25 17:38:16]

lgtm

[qiangchen, 2016-04-25 21:04:31]

The CQ bit was checked by qiangchen@chromium.org

[commit-bot: I haz the power, 2016-04-25 21:05:07]

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

[commit-bot: I haz the power, 2016-04-25 21:05:08]

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

[commit-bot: I haz the power, 2016-04-25 21:05:09]

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

[qiangchen, 2016-04-25 21:19:25]

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

Review URL: https://codereview.chromium.org/1895233003

Cr-Commit-Position: refs/heads/master@{#388635}
(cherry picked from commit 4089952731e1ad0a6cf83764042d59a41fbc42fe)
==========

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

Review URL: https://codereview.chromium.org/1895233003

Cr-Commit-Position: refs/heads/master@{#388635}
(cherry picked from commit 4089952731e1ad0a6cf83764042d59a41fbc42fe)

NOTRY=true
NOPRESUBMIT=true
==========

[qiangchen, 2016-04-25 21:19:31]

The CQ bit was checked by qiangchen@chromium.org

[commit-bot: I haz the power, 2016-04-25 21:20:16]

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

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

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

Review URL: https://codereview.chromium.org/1895233003

Cr-Commit-Position: refs/heads/master@{#388635}
(cherry picked from commit 4089952731e1ad0a6cf83764042d59a41fbc42fe)

NOTRY=true
NOPRESUBMIT=true
==========

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

Review URL: https://codereview.chromium.org/1895233003

Cr-Commit-Position: refs/heads/master@{#388635}
(cherry picked from commit 4089952731e1ad0a6cf83764042d59a41fbc42fe)

NOTRY=true
NOPRESUBMIT=true
==========

[commit-bot: I haz the power, 2016-04-25 21:22:13]

Committed patchset #1 (id:20001)