[iOS] Default initialize Settings::setFixedPositionCreatesStackingContext() to true
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 6 Oct 2013 00:34:04 +0000 (00:34 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 6 Oct 2013 00:34:04 +0000 (00:34 +0000)
https://bugs.webkit.org/show_bug.cgi?id=122338
<rdar://problem/14908151>

Reviewed by Darin Adler.

On iOS we always want an element with a fixed position to create a stacking context.
See <rdar://problem/9813262> for more details.

* page/Settings.cpp: Define constant defaultFixedPositionCreatesStackingContext to
be true when building for iOS and false otherwise.
* page/Settings.in: Initialize the setting fixedPositionCreatesStackingContext with
the value of the constant defaultFixedPositionCreatesStackingContext.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@156979 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/page/Settings.cpp
Source/WebCore/page/Settings.in

index 32d76ed..91414f4 100644 (file)
@@ -1,3 +1,19 @@
+2013-10-05  Daniel Bates  <dabates@apple.com>
+
+        [iOS] Default initialize Settings::setFixedPositionCreatesStackingContext() to true
+        https://bugs.webkit.org/show_bug.cgi?id=122338
+        <rdar://problem/14908151>
+
+        Reviewed by Darin Adler.
+
+        On iOS we always want an element with a fixed position to create a stacking context.
+        See <rdar://problem/9813262> for more details.
+
+        * page/Settings.cpp: Define constant defaultFixedPositionCreatesStackingContext to
+        be true when building for iOS and false otherwise.
+        * page/Settings.in: Initialize the setting fixedPositionCreatesStackingContext with
+        the value of the constant defaultFixedPositionCreatesStackingContext.
+
 2013-10-05  Sam Weinig  <sam@webkit.org>
 
         CTTE: Pass DocumentWriter around as a reference
index 3cb4d09..1d02c95 100644 (file)
@@ -113,10 +113,12 @@ static EditingBehaviorType editingBehaviorTypeForPlatform()
 }
 
 #if PLATFORM(IOS)
+static const bool defaultFixedPositionCreatesStackingContext = true;
 static const bool defaultMediaPlaybackAllowsInline = false;
 static const bool defaultMediaPlaybackRequiresUserGesture = true;
 static const bool defaultShouldRespectImageOrientation = true;
 #else
+static const bool defaultFixedPositionCreatesStackingContext = false;
 static const bool defaultMediaPlaybackAllowsInline = true;
 static const bool defaultMediaPlaybackRequiresUserGesture = false;
 static const bool defaultShouldRespectImageOrientation = false;
index bef6574..1f53f06 100644 (file)
@@ -155,7 +155,7 @@ deviceSupportsMouse initial=true
 # support for touch adjustment.
 touchAdjustmentEnabled initial=true
 
-fixedPositionCreatesStackingContext initial=false
+fixedPositionCreatesStackingContext initial=defaultFixedPositionCreatesStackingContext
 syncXHRInDocumentsEnabled initial=true
 cookieEnabled initial=true
 mediaEnabled initial=true