Accept 'allowfullscreen' in addition to 'webkitallowfullscreen'.
authorscheib@chromium.org <scheib@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 21 Feb 2013 00:32:26 +0000 (00:32 +0000)
committerscheib@chromium.org <scheib@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 21 Feb 2013 00:32:26 +0000 (00:32 +0000)
commit9fb7717774c4ca5aa2ac0f8f2866b9f70e4ec80c
tree40832365c75633818e3349693bfdd295ec98bf1b
parente58b782af13bf37a263a36100a266ccf0cb5750c
Accept 'allowfullscreen' in addition to 'webkitallowfullscreen'.
https://bugs.webkit.org/show_bug.cgi?id=110374

Reviewed by Adam Barth.

Source/WebCore:

The fullscreen API also includes a new HTML attribute for iframes
which permits its use. That attribute, 'allowfullcreen' is now
included in the HTML specification [1] and is stable enough
to remove the webkit prefix.

[1] http://www.whatwg.org/specs/web-apps/current-work/multipage/the-iframe-element.html#attr-iframe-allowfullscreen

Tests: fullscreen/full-screen-enabled-prefixed.html
       fullscreen/full-screen-iframe-allowed-prefixed.html
       fullscreen/full-screen-iframe-with-mixed-allow-webkitallow-attribute.html

* dom/Document.cpp:
(WebCore::isAttributeOnAllOwners):
(WebCore::Document::fullScreenIsAllowedForElement):
(WebCore::Document::webkitFullscreenEnabled):
* html/HTMLAttributeNames.in:

LayoutTests:

Two previous tests duplicated to preserve testing webkitallowfullscreen
and then modified to test allowfullscreen.

A new test 'full-screen-iframe-with-mixed-allow-webkitallow-attribute.html'
added to test a mixed nesting of iframs with both allowfullscreen and
webkitallowfullscreen.

* fullscreen/full-screen-enabled-prefixed-expected.txt: Added.
* fullscreen/full-screen-enabled-prefixed.html: Copied from LayoutTests/fullscreen/full-screen-enabled.html.
* fullscreen/full-screen-enabled.html:
* fullscreen/full-screen-iframe-allowed-prefixed-expected.txt: Added.
* fullscreen/full-screen-iframe-allowed-prefixed.html: Copied from LayoutTests/fullscreen/full-screen-iframe-allowed.html.
* fullscreen/full-screen-iframe-allowed.html:
* fullscreen/full-screen-iframe-with-mixed-allow-webkitallow-attribute-expected.txt:
* fullscreen/full-screen-iframe-with-mixed-allow-webkitallow-attribute.html: Added.
* fullscreen/resources/full-screen-iframe-with-mixed-allow-webkitallow-iframe1.html: Added.
* fullscreen/resources/full-screen-iframe-with-mixed-allow-webkitallow-iframe2.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@143533 268f45cc-cd09-0410-ab3c-d52691b4dbfc
14 files changed:
LayoutTests/ChangeLog
LayoutTests/fullscreen/full-screen-enabled-prefixed-expected.txt [new file with mode: 0644]
LayoutTests/fullscreen/full-screen-enabled-prefixed.html [new file with mode: 0644]
LayoutTests/fullscreen/full-screen-enabled.html
LayoutTests/fullscreen/full-screen-iframe-allowed-prefixed-expected.txt [new file with mode: 0644]
LayoutTests/fullscreen/full-screen-iframe-allowed-prefixed.html [new file with mode: 0644]
LayoutTests/fullscreen/full-screen-iframe-allowed.html
LayoutTests/fullscreen/full-screen-iframe-with-mixed-allow-webkitallow-attribute-expected.txt [new file with mode: 0644]
LayoutTests/fullscreen/full-screen-iframe-with-mixed-allow-webkitallow-attribute.html [new file with mode: 0644]
LayoutTests/fullscreen/resources/full-screen-iframe-with-mixed-allow-webkitallow-iframe1.html [new file with mode: 0644]
LayoutTests/fullscreen/resources/full-screen-iframe-with-mixed-allow-webkitallow-iframe2.html [new file with mode: 0644]
Source/WebCore/ChangeLog
Source/WebCore/dom/Document.cpp
Source/WebCore/html/HTMLAttributeNames.in