Fix an issue that webkit-transform might impact fullscreen video

Fix an issue that webkit-transform might impact fullscreen video

Fullscreen video should ignore all css transforms.

BUG=356282
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=170810

[qinmin, 2014-04-01 19:27:16]

PTAL

[qinmin, 2014-04-03 17:24:09]

ping. acolwell@, would you please take a look at this change? 

According to scherkus@ on the bug linked to this CL, we should ignore all webkit
transforms on fullscreen video element.

[acolwell GONE FROM CHROMIUM, 2014-04-03 17:44:33]

It seems like we should be able to make this a ref test instead of a pixel test.
This should look the same as fullscreen w/o the transform right?

https://codereview.chromium.org/221333002/diff/20001/LayoutTests/fullscreen/v...
File LayoutTests/fullscreen/video-webkit-transform.html (right):

https://codereview.chromium.org/221333002/diff/20001/LayoutTests/fullscreen/v...
LayoutTests/fullscreen/video-webkit-transform.html:12: waitForEvent(document,
'webkitfullscreenchange', function() {
nit: { on the next line.

https://codereview.chromium.org/221333002/diff/20001/LayoutTests/fullscreen/v...
LayoutTests/fullscreen/video-webkit-transform.html:16: runWithKeyDown(function()
{
ditto

[qinmin, 2014-04-03 18:35:35]

Also changed pixel test to ref test

https://codereview.chromium.org/221333002/diff/20001/LayoutTests/fullscreen/v...
File LayoutTests/fullscreen/video-webkit-transform.html (right):

https://codereview.chromium.org/221333002/diff/20001/LayoutTests/fullscreen/v...
LayoutTests/fullscreen/video-webkit-transform.html:12: waitForEvent(document,
'webkitfullscreenchange', function() {
On 2014/04/03 17:44:34, acolwell wrote:
> nit: { on the next line.

Done.

https://codereview.chromium.org/221333002/diff/20001/LayoutTests/fullscreen/v...
LayoutTests/fullscreen/video-webkit-transform.html:16: runWithKeyDown(function()
{
On 2014/04/03 17:44:34, acolwell wrote:
> ditto

Done.

[acolwell GONE FROM CHROMIUM, 2014-04-03 20:26:36]

lgtm

[qinmin, 2014-04-03 21:02:05]

The CQ bit was checked by qinmin@chromium.org

[commit-bot: I haz the power, 2014-04-03 21:02:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/qinmin@chromium.org/221333002/40001

[commit-bot: I haz the power, 2014-04-03 21:36:45]

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

[commit-bot: I haz the power, 2014-04-03 21:36:46]

Try jobs failed on following builders:
  tryserver.blink on linux_blink_rel

[qinmin, 2014-04-03 22:10:55]

The CQ bit was checked by qinmin@chromium.org

[commit-bot: I haz the power, 2014-04-03 22:11:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/qinmin@chromium.org/221333002/40001

[commit-bot: I haz the power, 2014-04-03 22:15:28]

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

[commit-bot: I haz the power, 2014-04-03 22:15:28]

Try jobs failed on following builders:
  tryserver.blink on linux_blink_dbg

[qinmin, 2014-04-03 22:53:50]

The CQ bit was checked by qinmin@chromium.org

[commit-bot: I haz the power, 2014-04-03 22:53:54]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/qinmin@chromium.org/221333002/60001

[commit-bot: I haz the power, 2014-04-03 23:19:09]

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

[commit-bot: I haz the power, 2014-04-03 23:19:10]

Try jobs failed on following builders:
  tryserver.blink on win_blink_rel

[qinmin, 2014-04-03 23:41:34]

The CQ bit was checked by qinmin@chromium.org

[commit-bot: I haz the power, 2014-04-03 23:41:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/qinmin@chromium.org/221333002/60001

[commit-bot: I haz the power, 2014-04-04 00:27:54]

Change committed as 170810