commit 7a9c45c01cadea5d8b3e9671a57311247a8c414d
author joshualitt <joshualitt@chromium.org> Tue May 26 12:32:23 2015 -0700
committer Commit bot <commit-bot@chromium.org> Tue May 26 12:32:23 2015 -0700
tree e49449b3a3c58ea8be52101ecff2c175d626475a
parent dc47ff7f1fe8fd56aed93a79cad679aa51914388

real fix for textblob use after gpu free

This patch addresses two issues:
1) Textblobs with abandoned strikes were not properly regenerating.
2) Provided by ericrk - GrTextBlobCache removed blobs from |fCache| during freeAll, but left
potentially released blobs in the |fBlobList|. We now remove these from
|fBlobList| as well.
BUG=skia:

Review URL: https://codereview.chromium.org/1160633002

gm/textblobuseaftergpufree.cpp

diff --git a/gm/textblobuseaftergpufree.cpp b/gm/textblobuseaftergpufree.cpp
new file mode 100644
index 0000000..127e436
--- /dev/null
+++ b/gm/textblobuseaftergpufree.cpp
@@ -0,0 +1,73 @@
+/*
+ * Copyright 2015 Google Inc.
+ *
+ * Use of this source code is governed by a BD-style license that can be
+ * found in the LICENSE file.
+ */
+
+#include "gm.h"
+
+#if SK_SUPPORT_GPU
+
+#include "SkCanvas.h"
+#include "SkSurface.h"
+#include "SkTextBlob.h"
+
+// This tests that we correctly regenerate textblobs after freeing all gpu resources crbug/491350
+namespace skiagm {
+class TextBlobUseAfterGpuFree : public GM {
+public:
+    TextBlobUseAfterGpuFree() { }
+
+protected:
+    SkString onShortName() override {
+        return SkString("textblobuseaftergpufree");
+    }
+
+    SkISize onISize() override {
+        return SkISize::Make(kWidth, kHeight);
+    }
+
+    void onDraw(SkCanvas* canvas) override {
+        // This GM exists to test a specific feature of the GPU backend.
+        if (NULL == canvas->getGrContext()) {
+            this->drawGpuOnlyMessage(canvas);
+            return;
+        }
+
+        const char text[] = "Hamburgefons";
+
+        SkPaint paint;
+        sk_tool_utils::set_portable_typeface(&paint);
+        paint.setTextSize(20);
+
+        SkTextBlobBuilder builder;
+
+        sk_tool_utils::add_to_text_blob(&builder, text, paint, 10, 10);
+
+        SkAutoTUnref<const SkTextBlob> blob(builder.build());
+
+        // draw textblob
+        SkRect rect = SkRect::MakeLTRB(0.f, 0.f, SkIntToScalar(kWidth), kHeight / 2.f);
+        SkPaint rectPaint;
+        rectPaint.setColor(0xffffffff);
+        canvas->drawRect(rect, rectPaint);
+        canvas->drawTextBlob(blob.get(), 10, 50, paint);
+
+        // This text should look fine
+        canvas->getGrContext()->freeGpuResources();
+        canvas->drawTextBlob(blob.get(), 10, 150, paint);
+    }
+
+private:
+    static const int kWidth = 200;
+    static const int kHeight = 200;
+
+    typedef GM INHERITED;
+};
+
+//////////////////////////////////////////////////////////////////////////////
+
+DEF_GM( return SkNEW(TextBlobUseAfterGpuFree); )
+}
+#endif

src/gpu/GrAtlasTextContext.cpp

diff --git a/src/gpu/GrAtlasTextContext.cpp b/src/gpu/GrAtlasTextContext.cpp
index 35a9164..750cb13 100644
--- a/src/gpu/GrAtlasTextContext.cpp
+++ b/src/gpu/GrAtlasTextContext.cpp
@@ -1592,7 +1592,8 @@
             TextInfo& info = run.fSubRunInfo[args.fSubRun];
 
             uint64_t currentAtlasGen = fFontCache->atlasGeneration(fMaskFormat);
-            bool regenerateTextureCoords = info.fAtlasGeneration != currentAtlasGen;
+            bool regenerateTextureCoords = info.fAtlasGeneration != currentAtlasGen ||
+                                           run.fStrike->isAbandoned();
             bool regenerateColors;
             if (fUseDistanceFields) {
                 regenerateColors = !fUseLCDText && run.fColor != args.fColor;

src/gpu/GrCommandBuilder.cpp

diff --git a/src/gpu/GrCommandBuilder.cpp b/src/gpu/GrCommandBuilder.cpp
index 6b56dea..28c0cf2 100644
--- a/src/gpu/GrCommandBuilder.cpp
+++ b/src/gpu/GrCommandBuilder.cpp
@@ -33,9 +33,7 @@
         rect = &r;
     }
 
-    SkASSERT(color == GrColor_ILLEGAL ||
-             canIgnoreRect ||
-             (rect->fLeft <= rect->fRight && rect->fTop <= rect->fBottom));
+    SkASSERT(canIgnoreRect || (rect->fLeft <= rect->fRight && rect->fTop <= rect->fBottom));
 
     Clear* clr = GrNEW_APPEND_TO_RECORDER(*this->cmdBuffer(), Clear, (renderTarget));
     GrColorIsPMAssert(color);

src/gpu/GrTextBlobCache.cpp

diff --git a/src/gpu/GrTextBlobCache.cpp b/src/gpu/GrTextBlobCache.cpp
index d66f432..6f17177 100644
--- a/src/gpu/GrTextBlobCache.cpp
+++ b/src/gpu/GrTextBlobCache.cpp
@@ -42,7 +42,9 @@
 void GrTextBlobCache::freeAll() {
     SkTDynamicHash<BitmapTextBlob, BitmapTextBlob::Key>::Iter iter(&fCache);
     while (!iter.done()) {
-        (&(*iter))->unref();
+        BitmapTextBlob* blob = &(*iter);
+        fBlobList.remove(blob);
+        blob->unref();
         ++iter;
     }
     fCache.rewind();