[webkit-reviews] review granted: [Bug 108314] [BlackBerry] Enable non-video element enter/exit fullscreen. : [Attachment 186749] Patch

[bugzilla-daemon at webkit.org]

Antonio Gomes <tonikitoo at webkit.org> has granted Mary Wu
<mary.wu at torchmobile.com.cn>'s request for review:
Bug 108314: [BlackBerry] Enable non-video element enter/exit fullscreen.
https://bugs.webkit.org/show_bug.cgi?id=108314

Attachment 186749: Patch
https://bugs.webkit.org/attachment.cgi?id=186749&action=review

------- Additional Comments from Antonio Gomes <tonikitoo at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=186749&action=review


- we have now another inconsistency: fullscreen x fullScreen. We could address
it as well in a follow up.

> Source/WebKit/blackberry/Api/WebPage.cpp:5122
> +    Element* element = static_cast<Element*>(d->m_fullscreenNode.get());

because you cast, please call 'm_fullscreenNode->isElement())