Bug 129707 - [iOS][MediaControls] Reverse-pinching using the built-in controls should enter full screen
Summary: [iOS][MediaControls] Reverse-pinching using the built-in controls should ente...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jer Noble
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-03-04 14:31 PST by Jer Noble
Modified: 2014-03-19 14:39 PDT (History)
6 users (show)

See Also:


Attachments
Patch (3.52 KB, patch)
2014-03-04 14:35 PST, Jer Noble
eric.carlson: review+
Details | Formatted Diff | Diff
Patch for landing (3.22 KB, patch)
2014-03-10 10:20 PDT, Jer Noble
commit-queue: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jer Noble 2014-03-04 14:31:59 PST
[iOS][MediaControls] Reverse-pinching using the built-in controls should enter full screen
Comment 1 Jer Noble 2014-03-04 14:35:08 PST
Created attachment 225817 [details]
Patch
Comment 2 Eric Carlson 2014-03-04 14:42:01 PST
Comment on attachment 225817 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=225817&action=review

> Source/WebCore/Modules/mediacontrols/mediaControlsiOS.js:265
> +    handlePlayButtonTouchEnd: function(event) { 

Garbage character?
Comment 3 Jer Noble 2014-03-10 10:20:52 PDT
Created attachment 226316 [details]
Patch for landing
Comment 4 WebKit Commit Bot 2014-03-14 10:26:41 PDT
Comment on attachment 226316 [details]
Patch for landing

Rejecting attachment 226316 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-01', 'validate-changelog', '--check-oops', '--non-interactive', 226316, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

ChangeLog entry in Source/WebCore/ChangeLog contains OOPS!.

Full output: http://webkit-queues.appspot.com/results/6579497612607488
Comment 5 Jer Noble 2014-03-19 14:39:25 PDT
Committed r165919: <http://trac.webkit.org/changeset/165919>