Compensate for canvas-side vertical text translation in bounding box computation

bug:7267294
Change-Id: Ia6de9264b6c3680707b8fb8833d69d7cc7e850e8

4 files changed, 6 insertions, 0 deletions

diff --git a/Source/WebCore/platform/graphics/android/context/PlatformGraphicsContext.h b/Source/WebCore/platform/graphics/android/context/PlatformGraphicsContext.h
index 56aceff..14537c8 100644
--- a/Source/WebCore/platform/graphics/android/context/PlatformGraphicsContext.h
+++ b/Source/WebCore/platform/graphics/android/context/PlatformGraphicsContext.h
@@ -148,6 +148,7 @@ public:
                                  const IntRect& thumb = IntRect()) = 0;
 
     virtual SkCanvas* recordingCanvas() = 0;
+    virtual void setTextOffset(FloatSize offset) = 0;
 
     void setRawState(State* state) { m_state = state; }
 
diff --git a/Source/WebCore/platform/graphics/android/context/PlatformGraphicsContextRecording.cpp b/Source/WebCore/platform/graphics/android/context/PlatformGraphicsContextRecording.cpp
index 867ff56..8c87cfb 100644
--- a/Source/WebCore/platform/graphics/android/context/PlatformGraphicsContextRecording.cpp
+++ b/Source/WebCore/platform/graphics/android/context/PlatformGraphicsContextRecording.cpp
@@ -914,6 +914,8 @@ void PlatformGraphicsContextRecording::drawPosText(const void* inText, size_t by
         return;
     }
     FloatRect bounds = approximateTextBounds(byteLength / sizeof(uint16_t), inPos, inPaint);
+    bounds.move(m_textOffset); // compensate font rendering-side translates
+
     const SkPaint* paint = mRecording->recording()->getSkPaint(inPaint);
     size_t posSize = sizeof(SkPoint) * paint->countText(inText, byteLength);
     void* text = heap()->alloc(byteLength);
diff --git a/Source/WebCore/platform/graphics/android/context/PlatformGraphicsContextRecording.h b/Source/WebCore/platform/graphics/android/context/PlatformGraphicsContextRecording.h
index 061ee0e..b120ac3 100644
--- a/Source/WebCore/platform/graphics/android/context/PlatformGraphicsContextRecording.h
+++ b/Source/WebCore/platform/graphics/android/context/PlatformGraphicsContextRecording.h
@@ -64,6 +64,7 @@ public:
     virtual bool isPaintingDisabled();
 
     virtual SkCanvas* recordingCanvas();
+    virtual void setTextOffset(FloatSize offset) { m_textOffset = offset; }
 
     virtual ContextType type() { return RecordingContext; }
 
@@ -209,6 +210,7 @@ private:
     bool m_hasText;
     bool m_isEmpty;
     RecordingContextCanvasProxy m_canvasProxy;
+    FloatSize m_textOffset;
 };
 
 }
diff --git a/Source/WebCore/platform/graphics/android/context/PlatformGraphicsContextSkia.h b/Source/WebCore/platform/graphics/android/context/PlatformGraphicsContextSkia.h
index 6bf53d7..2c1d8cb 100644
--- a/Source/WebCore/platform/graphics/android/context/PlatformGraphicsContextSkia.h
+++ b/Source/WebCore/platform/graphics/android/context/PlatformGraphicsContextSkia.h
@@ -39,6 +39,7 @@ public:
 
     virtual ContextType type() { return PaintingContext; }
     virtual SkCanvas* recordingCanvas() { return mCanvas; }
+    virtual void setTextOffset(FloatSize offset) {}
 
     // FIXME: This is used by ImageBufferAndroid, which should really be
     //        managing the canvas lifecycle itself