Address the review comment, missed from the previous push
authorEhsan Akhgari <ehsan@mozilla.com>
Wed, 22 Feb 2012 17:15:53 -0500
changeset 89222 6b4f7dee6fe86943d20108b920e90ff550d91fa6
parent 89221 e9930e885ef793055b84d6889c4bc18838a56b52
child 89223 c745d456c19bd83e5e17fbec50972e4be358e22f
push id22242
push userkgupta@mozilla.com
push dateWed, 14 Mar 2012 15:19:09 +0000
treeherdermozilla-central@936ef50fa498 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
milestone13.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Address the review comment, missed from the previous push
mobile/android/base/gfx/GeckoLayerClient.java
--- a/mobile/android/base/gfx/GeckoLayerClient.java
+++ b/mobile/android/base/gfx/GeckoLayerClient.java
@@ -130,17 +130,17 @@ public class GeckoLayerClient implements
     public Rect beginDrawing(int width, int height, int tileWidth, int tileHeight,
                              String metadata, boolean hasDirectTexture) {
         Log.e(LOGTAG, "### beginDrawing " + width + " " + height + " " + tileWidth + " " +
               tileHeight + " " + hasDirectTexture);
 
         // If the viewport has changed but we still don't have the latest viewport
         // from Gecko, ignore the viewport passed to us from Gecko since that is going
         // to be wrong.
-        if (!mFirstPaint && mIgnorePaintsPendingViewportSizeChange && !mUpdateViewportOnEndDraw) {
+        if (!mFirstPaint && mIgnorePaintsPendingViewportSizeChange) {
             return null;
         }
         mFirstPaint = false;
 
         // If we've changed surface types, cancel this draw
         if (handleDirectTextureChange(hasDirectTexture)) {
             Log.e(LOGTAG, "### Cancelling draw due to direct texture change");
             return null;