Update AVKit usage of pip, with additional cleanup in preferences rdar://problem/21386853
Uploading several sub-patches for review, and one large one that is the combination of all of them, for EWS to chew on.
Created attachment 255062 [details] Refactor allowsOptimizedFullscreen, set bool with member instead of local var.
Created attachment 255063 [details] Update delegate calls
Created attachment 255064 [details] Replace start/stop PiP
Created attachment 255065 [details] isPiPPossible
Created attachment 255066 [details] prepareForPictureInPictureStopWithCompletionHandler
Created attachment 255067 [details] enterFullscreenOptimized
Created attachment 255068 [details] remove unused setIsOptimized
Created attachment 255069 [details] Change media player enum to VideoFullscreenModePictureInPicture
Created attachment 255070 [details] Remove unused AVPlayerViewControllerOptimizedFullscreenStopReason
Created attachment 255071 [details] Replace wkIsOptimizedFullscreenSupported
Created attachment 255072 [details] Replace AVPlayerLayer setEnterOptimizedFullscreenModeEnabled
Created attachment 255073 [details] Update local variable names
Created attachment 255074 [details] Update mayAutomaticallyShowVideoOptimized
Created attachment 255075 [details] Replace _isShowingVideoOptimized and _mayAutomaticallyShowVideoOptimized
Created attachment 255076 [details] Update setting to allowsPictureInPictureMediaPlayback
Created attachment 255077 [details] Update MediaElementSession::allowsAlternateFullscreen
Created attachment 255078 [details] Add WebPreferences allowsPictureInPictureMediaPlayback; refactor alternate fullscreen to use that. Later patch will remove the preference since it is being used by another framework
Created attachment 255079 [details] Update shouldAllowAlternateFullscreen
Created attachment 255090 [details] Patch for submission
*** Bug 146094 has been marked as a duplicate of this bug. ***
19 r? in one bug report? :o I'm not sure if it is a good idea. The latest patch aggregates the previous smaller patch, am I right? Please remove the r? flags and set the obsolete flags on the other patches.
(In reply to comment #1) > Uploading several sub-patches for review, and one large one that is the > combination of all of them, for EWS to chew on. It's not good at all, the common way is to upload only one patch per bug report. Additionally a patch without changelog entry is insufficient.
Comment on attachment 255090 [details] Patch for submission Did you miss an instance in Settings.in?
(In reply to comment #24) > Comment on attachment 255090 [details] > Patch for submission > > Did you miss an instance in Settings.in? No, this is another case where the builders don't update the generated settings files appropriately. I ran into this before when renaming the other media settings. It builds fine on my machine; I'll keep an eye on the bots.
I am going to check in a version of the patch that does not include the update to WKSI. That call should be removed. I'll file another bug to track that.
Created attachment 255142 [details] Patch
Created attachment 255149 [details] Patch for submission
Committed r185727: <http://trac.webkit.org/changeset/185727>