Title: [207596] branches/safari-602-branch/Source/WebCore
Revision
207596
Author
matthew_han...@apple.com
Date
2016-10-20 02:59:19 -0700 (Thu, 20 Oct 2016)

Log Message

Merge r205274. rdar://problem/28476952

Modified Paths

Diff

Modified: branches/safari-602-branch/Source/WebCore/ChangeLog (207595 => 207596)


--- branches/safari-602-branch/Source/WebCore/ChangeLog	2016-10-20 09:59:15 UTC (rev 207595)
+++ branches/safari-602-branch/Source/WebCore/ChangeLog	2016-10-20 09:59:19 UTC (rev 207596)
@@ -1,5 +1,24 @@
 2016-10-20  Matthew Hanson  <matthew_han...@apple.com>
 
+        Merge r205274. rdar://problem/28476952
+
+    2016-08-31  Ricky Mondello  <rmonde...@apple.com>
+
+            Enable the YouTube Flash plug-in replacement behavior on all Cocoa ports
+            https://bugs.webkit.org/show_bug.cgi?id=161453
+            <rdar://problem/28050847>
+
+            Reviewed by Eric Carlson.
+
+            Now that we have some tests for the URL transformation logic (r205212) and the ability to enable the YouTube
+            Flash plug-in replacement behavior independently from the QuickTime plug-in replacement behavior (r205214 and
+            r205271), enable the YouTube Flash plug-in replacement behavior for Cocoa ports. We can and will continue to
+            improve it.
+
+            * page/Settings.cpp: Enable the feature for PLATFORM(COCOA), rather than just PLATFORM(IOS).
+
+2016-10-20  Matthew Hanson  <matthew_han...@apple.com>
+
         Merge r205271. rdar://problem/28476952
 
     2016-08-31  Ricky Mondello  <rmonde...@apple.com>

Modified: branches/safari-602-branch/Source/WebCore/page/Settings.cpp (207595 => 207596)


--- branches/safari-602-branch/Source/WebCore/page/Settings.cpp	2016-10-20 09:59:15 UTC (rev 207595)
+++ branches/safari-602-branch/Source/WebCore/page/Settings.cpp	2016-10-20 09:59:19 UTC (rev 207596)
@@ -133,6 +133,12 @@
     ;
 }
 
+#if PLATFORM(COCOA)
+static const bool defaultYouTubeFlashPluginReplacementEnabled = true;
+#else
+static const bool defaultYouTubeFlashPluginReplacementEnabled = false;
+#endif
+
 #if PLATFORM(IOS)
 static const bool defaultFixedPositionCreatesStackingContext = true;
 static const bool defaultFixedBackgroundsPaintRelativeToDocument = true;
@@ -147,7 +153,6 @@
 static const bool defaultScrollingTreeIncludesFrames = true;
 static const bool defaultMediaControlsScaleWithPageZoom = true;
 static const bool defaultQuickTimePluginReplacementEnabled = true;
-static const bool defaultYouTubeFlashPluginReplacementEnabled = true;
 #else
 static const bool defaultFixedPositionCreatesStackingContext = false;
 static const bool defaultFixedBackgroundsPaintRelativeToDocument = false;
@@ -162,7 +167,6 @@
 static const bool defaultScrollingTreeIncludesFrames = false;
 static const bool defaultMediaControlsScaleWithPageZoom = true;
 static const bool defaultQuickTimePluginReplacementEnabled = false;
-static const bool defaultYouTubeFlashPluginReplacementEnabled = false;
 #endif
 
 static const bool defaultAllowsPictureInPictureMediaPlayback = true;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to