commit c3269aea5ba61a0c2228b029c7626802474825f4
author Jim Van Verth <jvanverth@google.com> Thu Sep 28 15:04:00 2017 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Thu Sep 28 19:38:42 2017 +0000
tree d568b551ab3382d2c9214451196c321f4dd797b4
parent 4e66d35fc38d45040827478f41e731e62ef75e0a

Miscellaneous dynamic atlas fixes.

Make Chinese fling sample closer to Android test.
Fix a bug in GrDrawOpAtlas::compact(), where the atlas
generation wasn't incremented so the client didn't know it changed.
Add some debug info for GrDrawOpAtlas.
Add a comment in GrDrawOpAtlas::setLastUseTokenBulk.

Change-Id: I79192a017870541a79731b1a22f665ec5deeff09
Reviewed-on: https://skia-review.googlesource.com/52761
Commit-Queue: Jim Van Verth <jvanverth@google.com>
Reviewed-by: Robert Phillips <robertphillips@google.com>

samplecode/SampleChineseFling.cpp

diff --git a/samplecode/SampleChineseFling.cpp b/samplecode/SampleChineseFling.cpp
index 036f834..536f2c2 100644
--- a/samplecode/SampleChineseFling.cpp
+++ b/samplecode/SampleChineseFling.cpp
@@ -25,13 +25,6 @@
   paint->setTextEncoding(SkPaint::kUTF32_TextEncoding);
 }
 
-static void get_unicode_row(SkUnichar base, SkUnichar glyphs[16]) {
-    for (int i = 0x0; i <= 0xF; ++i) {
-        glyphs[i] = base;
-        glyphs[i] |= i;
-    }
-}
-
 static sk_sp<SkTypeface> chinese_typeface() {
 #ifdef SK_BUILD_FOR_ANDROID
     return MakeResourceAsTypeface("/fonts/NotoSansCJK-Regular.ttc");
@@ -50,7 +43,7 @@
 
 class ChineseFlingView : public SampleView {
 public:
-    ChineseFlingView() {}
+    ChineseFlingView() : fBlobs(kNumBlobs) {}
 
 protected:
     bool onQuery(SkEvent* evt) override {
@@ -72,19 +65,28 @@
         SkPaint paint;
         make_paint(&paint, fTypeface);
 
+        // draw a consistent run of the 'words' - one word per line
+        int index = fIndex;
         for (SkScalar y = 0.0f; y < 1024.0f; ) {
-            int index = fRand.nextRangeU(0, fBlobs.count()-1);
 
             y += -fMetrics.fAscent;
             canvas->drawTextBlob(fBlobs[index], 0, y, paint);
 
             y += fMetrics.fDescent + fMetrics.fLeading;
+            ++index;
+            index %= fBlobs.count();
         }
+        // now "fling" a random amount
+        fIndex += fRand.nextRangeU(5, 20);
+        fIndex %= fBlobs.count();
 
         this->inval(nullptr);
     }
 
 private:
+    static constexpr auto kNumBlobs = 200;
+    static constexpr auto kWordLength = 16;
+
     void init() {
         fTypeface = chinese_typeface();
 
@@ -93,19 +95,25 @@
 
         paint.getFontMetrics(&fMetrics);
 
-        SkUnichar glyphs[16];
-
-        for (int32_t i = 0x4F00; i < 0x9FA0; i += 0x10) {
-
-            get_unicode_row(i, glyphs);
+        SkUnichar glyphs[kWordLength];
+        for (int32_t i = 0; i < kNumBlobs; ++i) {
+            this->createRandomWord(glyphs);
 
             SkTextBlobBuilder builder;
-
-            sk_tool_utils::add_to_text_blob_w_len(&builder, (const char*) glyphs, 16*4, paint,
-                                                  0, 0);
+            sk_tool_utils::add_to_text_blob_w_len(&builder, (const char*) glyphs, kWordLength*4,
+                                                  paint, 0, 0);
 
             fBlobs.emplace_back(builder.make());
         }
+
+        fIndex = 0;
+    }
+
+    // Construct a random kWordLength character 'word' drawing from the full Chinese set
+    void createRandomWord(SkUnichar glyphs[kWordLength]) {
+        for (int i = 0; i < kWordLength; ++i) {
+            glyphs[i] = fRand.nextRangeU(0x4F00, 0x9FA0);
+        }
     }
 
     bool                        fInitialized = false;
@@ -113,6 +121,7 @@
     SkPaint::FontMetrics        fMetrics;
     SkTArray<sk_sp<SkTextBlob>> fBlobs;
     SkRandom                    fRand;
+    int                         fIndex;
 
     typedef SkView INHERITED;
 };