Commit d6df5b5f authored by L. David Baron's avatar L. David Baron
Browse files

Bug 1287011 - Remove preference gfx.filter.nearest.force-enabled. r=gw280 

This preference was added in bug 747274, and the need for it (i.e.,
having it default to a different value on Android) was removed a few
weeks later in bug 750598.

Whether we honor the preference depends on which drawing function we go
through, which bug 720531 will make even more inconsistent than it
already is (by causing it to be honored for border-image-repeat: space,
but not other values of border-image-repeat).

Rather than trying to clean it up, it seems best to remove it, since its
initial purpose appears to have been having a different default on
Android, which we longer need.

MozReview-Commit-ID: IFW2Ukwx4fH

--HG--
extra : transplant_source : q1%22Ad%D2%AF%8E%E5%E2%257C%A3%9F%95%DB%A5%92%B2
parent 4be29e46

layout/base/nsLayoutUtils.cpp

+0 −19
Original line number Diff line number Diff line
@@ -638,21 +638,6 @@ nsLayoutUtils::IsAnimationLoggingEnabled() 
  return sShouldLog;

}



bool

nsLayoutUtils::UseBackgroundNearestFiltering()

{

  static bool sUseBackgroundNearestFilteringEnabled;

  static bool sUseBackgroundNearestFilteringPrefInitialised = false;



  if (!sUseBackgroundNearestFilteringPrefInitialised) {

    sUseBackgroundNearestFilteringPrefInitialised = true;

    sUseBackgroundNearestFilteringEnabled =

      Preferences::GetBool("gfx.filter.nearest.force-enabled", false);

  }



  return sUseBackgroundNearestFilteringEnabled;

}



bool

nsLayoutUtils::GPUImageScalingEnabled()

{
@@ -6951,10 +6936,6 @@ nsLayoutUtils::DrawBackgroundImage(gfxContext& aContext, 
  PROFILER_LABEL("layout", "nsLayoutUtils::DrawBackgroundImage",

                 js::ProfileEntry::Category::GRAPHICS);



  if (UseBackgroundNearestFiltering()) {

    aSamplingFilter = SamplingFilter::POINT;

  }



  SVGImageContext svgContext(aImageSize, Nothing());



  /* Fast path when there is no need for image spacing */

layout/base/nsLayoutUtils.h

+0 −6
Original line number Diff line number Diff line
@@ -2286,12 +2286,6 @@ public: 
                                                  const nsSize& aVisibleSize,

                                                  const nsSize& aDisplaySize);



  /**

   * Checks if we should forcibly use nearest pixel filtering for the

   * background.

   */

  static bool UseBackgroundNearestFiltering();



  /**

   * Checks whether we want to use the GPU to scale images when

   * possible.

modules/libpref/init/all.js

+0 −2
Original line number Diff line number Diff line
@@ -708,8 +708,6 @@ pref("gfx.bundled_fonts.force-enabled", false); 
// whether to try and do something about it (e.g. download additional fonts)?

pref("gfx.missing_fonts.notify", false);



pref("gfx.filter.nearest.force-enabled", false);



// prefs controlling the font (name/cmap) loader that runs shortly after startup

pref("gfx.font_loader.families_per_slice", 3); // read in info 3 families at a time

#ifdef XP_WIN