[webkit-reviews] review granted: [Bug 117556] "Open Video in New Window" doesn't make sense in fullscreen : [Attachment 204496] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jun 12 12:35:14 PDT 2013


Beth Dakin <bdakin at apple.com> has granted Ruth Fong <ruthiecftg at gmail.com>'s
request for review:
Bug 117556: "Open Video in New Window" doesn't make sense in fullscreen
https://bugs.webkit.org/show_bug.cgi?id=117556

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

------- Additional Comments from Beth Dakin <bdakin at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=204496&action=review


> Source/WebKit2/ChangeLog:14
> +	   Hooks into new InjectedBundleHitTestResult::mediaIsInFullscreen()
method.

This is a nitpick, but in the future, you should add some newlines in this
section of the ChangeLog to make it a little more readable.


More information about the webkit-reviews mailing list