Fix drawPosText() bounds bug.

tests/RecordDrawTest.cpp

 /*
  * Copyright 2014 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "Test.h"
 #include "RecordTestUtils.h"
 
@@ skipping 99 matching lines @@
     virtual void rewindInserts() SK_OVERRIDE {}
     virtual int getDepth() const SK_OVERRIDE { return -1; }
 
     struct Entry {
         uintptr_t data;
         SkRect bounds;
     };
     SkTDArray<Entry> entries;
 };
 
+// Like a==b, with a little slop recognizing that float equality can be weird.
+static bool sloppy_rect_eq(SkRect a, SkRect b) {
+    SkRect inset(a), outset(a);
+    inset.inset(1, 1);
+    outset.outset(1, 1);
+    return outset.contains(b) && !inset.contains(b);
+}
+
 // This test is not meant to make total sense yet.  It's testing the status quo
 // of SkRecordFillBounds(), which itself doesn't make total sense yet.
 DEF_TEST(RecordDraw_BBH, r) {
-    TestBBH bbh;
-
     SkRecord record;
-
     SkRecorder recorder(&record, W, H);
     recorder.save();
         recorder.clipRect(SkRect::MakeWH(400, 500));
         recorder.scale(2, 2);
         recorder.drawRect(SkRect::MakeWH(320, 240), SkPaint());
     recorder.restore();
 
+    TestBBH bbh;
     SkRecordFillBounds(record, &bbh);
 
     REPORTER_ASSERT(r, bbh.entries.count() == 5);
     for (int i = 0; i < bbh.entries.count(); i++) {
         REPORTER_ASSERT(r, bbh.entries[i].data == (uintptr_t)i);
 
-        // We'd like to assert bounds == SkRect::MakeWH(400, 480).
-        // But we allow a little slop in recognition that float equality can be weird.
-        REPORTER_ASSERT(r,  SkRect::MakeLTRB(-1, -1, 401, 481).contains(bbh.entries[i].bounds));
-        REPORTER_ASSERT(r, !SkRect::MakeLTRB(+1, +1, 399, 479).contains(bbh.entries[i].bounds));
+        REPORTER_ASSERT(r, sloppy_rect_eq(SkRect::MakeWH(400, 480), bbh.entries[i].bounds));
     }
 }
 
+// A regression test for crbug.com/409110.
+DEF_TEST(RecordDraw_TextBounds, r) {
+    SkRecord record;
+    SkRecorder recorder(&record, W, H);
+
+    // Two Chinese characters in UTF-8.
+    const char text[] = { '\xe6', '\xbc', '\xa2', '\xe5', '\xad', '\x97' };
+    const size_t bytes = SK_ARRAY_COUNT(text);
+
+    const SkScalar xpos[] = { 10, 20 };
+    recorder.drawPosTextH(text, bytes, xpos, 30, SkPaint());
+
+    const SkPoint pos[] = { {40, 50}, {60, 70} };
+    recorder.drawPosText(text, bytes, pos, SkPaint());
+
+    TestBBH bbh;
+    SkRecordFillBounds(record, &bbh);
+    REPORTER_ASSERT(r, bbh.entries.count() == 2);
+
+    // We can make these next assertions confidently because SkRecordFillBounds
+    // builds its bounds by overestimating font metrics in a platform-independent way.
+    // If that changes, these tests will need to be more flexible.
+    REPORTER_ASSERT(r, sloppy_rect_eq(bbh.entries[0].bounds, SkRect::MakeLTRB(-86,  6, 116, 54)));
+    REPORTER_ASSERT(r, sloppy_rect_eq(bbh.entries[1].bounds, SkRect::MakeLTRB(-56, 26, 156, 94)));
+}
+
 // Base test to ensure start/stop range is respected
 DEF_TEST(RecordDraw_PartialStartStop, r) {
     static const int kWidth = 10, kHeight = 10;
 
     SkRect r1 = { 0, 0, kWidth,   kHeight };
     SkRect r2 = { 0, 0, kWidth,   kHeight/2 };
     SkRect r3 = { 0, 0, kWidth/2, kHeight };
     SkPaint p;
 
     SkRecord record;
@@ skipping 31 matching lines @@
 
     REPORTER_ASSERT(r, 3 == rerecord.count());
     assert_type<SkRecords::Save>    (r, rerecord, 0);
     assert_type<SkRecords::DrawRect>(r, rerecord, 1);
     assert_type<SkRecords::Restore> (r, rerecord, 2);
 
     const SkRecords::DrawRect* drawRect = assert_type<SkRecords::DrawRect>(r, rerecord, 1);
     REPORTER_ASSERT(r, drawRect->rect == rect);
     REPORTER_ASSERT(r, drawRect->paint.getColor() == SK_ColorRED);
 }