Backed out changeset a691b474af0c (bug 1501804) for failing Android at org.mozilla.geckoview.test.crash.CrashTest.crashParent a=backout
authorRazvan Maries <rmaries@mozilla.com>
Mon, 12 Nov 2018 18:23:47 +0200
changeset 498456 d74a918ff930da31a1ef3ad019ae27fdc75b0d44
parent 498455 a4175191b29b274d59ed117567ec03df9d6b8cf5
child 498457 6226ce7f4038d566fe750b1d13f107146d137463
push id10167
push userrmaries@mozilla.com
push dateMon, 12 Nov 2018 16:24:23 +0000
treeherdermozilla-beta@d74a918ff930 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbackout
bugs1501804
milestone64.0
backs outa691b474af0cac5d467179f06a9f07c37a3eba44
Backed out changeset a691b474af0c (bug 1501804) for failing Android at org.mozilla.geckoview.test.crash.CrashTest.crashParent a=backout
mobile/android/geckoview/src/main/java/org/mozilla/geckoview/GeckoRuntimeSettings.java
--- a/mobile/android/geckoview/src/main/java/org/mozilla/geckoview/GeckoRuntimeSettings.java
+++ b/mobile/android/geckoview/src/main/java/org/mozilla/geckoview/GeckoRuntimeSettings.java
@@ -114,17 +114,17 @@ public final class GeckoRuntimeSettings 
         /**
          * Set whether support for web fonts should be enabled.
          *
          * @param flag A flag determining whether web fonts should be enabled.
          *             Default is true.
          * @return This Builder instance.
          */
         public @NonNull Builder webFontsEnabled(final boolean flag) {
-            mSettings.mWebFonts.set(flag ? 1 : 0);
+            mSettings.mWebFonts.set(flag);
             return this;
         }
 
         /**
          * Set whether there should be a pause during startup. This is useful if you need to
          * wait for a debugger to attach.
          *
          * @param enabled A flag determining whether there will be a pause early in startup.
@@ -342,18 +342,18 @@ public final class GeckoRuntimeSettings 
             }
         }
     }
 
     /* package */ Pref<Boolean> mJavaScript = new Pref<Boolean>(
         "javascript.enabled", true);
     /* package */ Pref<Boolean> mRemoteDebugging = new Pref<Boolean>(
         "devtools.debugger.remote-enabled", false);
-    /* package */ Pref<Integer> mWebFonts = new Pref<Integer>(
-        "browser.display.use_document_fonts", 1);
+    /* package */ Pref<Boolean> mWebFonts = new Pref<Boolean>(
+        "browser.display.use_document_fonts", true);
     /* package */ Pref<Integer> mCookieBehavior = new Pref<Integer>(
         "network.cookie.cookieBehavior", COOKIE_ACCEPT_ALL);
     /* package */ Pref<Integer> mCookieLifetime = new Pref<Integer>(
         "network.cookie.lifetimePolicy", COOKIE_LIFETIME_NORMAL);
     /* package */ Pref<String> mTrackingProtection = new Pref<String>(
         "urlclassifier.trackingTable",
         TrackingProtection.buildPrefValue(
             TrackingProtectionDelegate.CATEGORY_TEST |
@@ -494,27 +494,27 @@ public final class GeckoRuntimeSettings 
     }
 
     /**
      * Get whether web fonts support is enabled.
      *
      * @return Whether web fonts support is enabled.
      */
     public boolean getWebFontsEnabled() {
-        return mWebFonts.get() != 0 ? true : false;
+        return mWebFonts.get();
     }
 
     /**
      * Set whether support for web fonts should be enabled.
      *
      * @param flag A flag determining whether web fonts should be enabled.
      * @return This GeckoRuntimeSettings instance.
      */
     public @NonNull GeckoRuntimeSettings setWebFontsEnabled(final boolean flag) {
-        mWebFonts.set(flag ? 1 : 0);
+        mWebFonts.set(flag);
         return this;
     }
 
     /**
      * Gets whether the pause-for-debugger is enabled or not.
      *
      * @return True if the pause is enabled.
      */