Bug 1434925 - Skip drawing hidden toolbar elements; r?rbarker draft
authorJim Chen <nchen@mozilla.com>
Thu, 08 Feb 2018 13:55:15 -0500
changeset 752658 fc5f73c9d831118188894624b50cfd3ac4e83722
parent 752655 22a2b0f1b1ab6054de76a92ed4a85d4d09e1514f
push id98335
push userbmo:nchen@mozilla.com
push dateThu, 08 Feb 2018 18:55:34 +0000
reviewersrbarker
bugs1434925
milestone60.0a1
Bug 1434925 - Skip drawing hidden toolbar elements; r?rbarker Apparently a toolbar element can be zero-sized when we try to draw it to the bitmap. We should just skip drawing in that case. MozReview-Commit-ID: LlFQX5uN20h
mobile/android/base/java/org/mozilla/gecko/toolbar/CanvasDelegate.java
--- a/mobile/android/base/java/org/mozilla/gecko/toolbar/CanvasDelegate.java
+++ b/mobile/android/base/java/org/mozilla/gecko/toolbar/CanvasDelegate.java
@@ -30,16 +30,20 @@ class CanvasDelegate {
 
         // DST_IN masks, DST_OUT clips.
         mMode = new PorterDuffXfermode(mode);
 
         mPaint = paint;
     }
 
     void draw(Canvas canvas, Path path, int width, int height) {
+        if (width <= 0 || height <= 0) {
+            return;
+        }
+
         Bitmap offscreen = Bitmap.createBitmap(width, height, Bitmap.Config.ARGB_8888);
         Canvas offscreenCanvas = new Canvas(offscreen);
 
         // Do a default draw.
         mDrawManager.defaultDraw(offscreenCanvas);
 
         if (path != null && !path.isEmpty()) {
             // ICS added double-buffering, which made it easier for drawing the Path directly over the DST.