[chromium] Use const SkBitmap& parameter for WebLayerTreeView::setFontAtlas
authorjamesr@google.com <jamesr@google.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 5 Nov 2012 20:49:05 +0000 (20:49 +0000)
committerjamesr@google.com <jamesr@google.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 5 Nov 2012 20:49:05 +0000 (20:49 +0000)
https://bugs.webkit.org/show_bug.cgi?id=101016

Reviewed by Adrienne Walker.

Passing SkBitmap by value requires including SkBitmap.h, which is tricky for some clients to do since it
requires having the rest of skia's headers on the include path and setting up the correct config to set
preprocessor defines.  Also shuffles the parameter order around to make rolling easier since you can't override
just by const ref-ness.

* chromium/public/WebLayerTreeView.h:
(WebKit::WebLayerTreeView::setFontAtlas):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@133516 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/Platform/ChangeLog
Source/Platform/chromium/public/WebLayerTreeView.h
Source/WebKit/chromium/src/WebViewImpl.cpp

index 05bfd9d..4771f13 100644 (file)
@@ -1,3 +1,18 @@
+2012-11-05  James Robinson  <jamesr@chromium.org>
+
+        [chromium] Use const SkBitmap& parameter for WebLayerTreeView::setFontAtlas
+        https://bugs.webkit.org/show_bug.cgi?id=101016
+
+        Reviewed by Adrienne Walker.
+
+        Passing SkBitmap by value requires including SkBitmap.h, which is tricky for some clients to do since it
+        requires having the rest of skia's headers on the include path and setting up the correct config to set
+        preprocessor defines.  Also shuffles the parameter order around to make rolling easier since you can't override
+        just by const ref-ness.
+
+        * chromium/public/WebLayerTreeView.h:
+        (WebKit::WebLayerTreeView::setFontAtlas):
+
 2012-11-05  Alok Priyadarshi  <alokp@chromium.org>
 
         [chromium] Pass canPaintLCDText to WebContentLayerClient::paintContents
index ff6d425..345045a 100644 (file)
@@ -26,7 +26,6 @@
 #ifndef WebLayerTreeView_h
 #define WebLayerTreeView_h
 
-#include "SkBitmap.h"
 #include "WebColor.h"
 #include "WebCommon.h"
 #include "WebFloatPoint.h"
@@ -34,6 +33,8 @@
 #include "WebPrivateOwnPtr.h"
 #include "WebSize.h"
 
+class SkBitmap;
+
 namespace WebKit {
 class WebGraphicsContext3D;
 class WebLayer;
@@ -170,7 +171,7 @@ public:
 
     // Provides a font atlas to use for debug visualizations. The atlas must be a bitmap containing glyph data, a table of
     // ASCII character values to a subrectangle of the atlas representing the corresponding glyph, and the glyph height.
-    virtual void setFontAtlas(SkBitmap, WebRect asciiToRectTable[128], int fontHeight) = 0;
+    virtual void setFontAtlas(WebRect asciiToRectTable[128], const SkBitmap&, int fontHeight) { }
 
     // Simulates a lost context. For testing only.
     virtual void loseCompositorContext(int numTimes) = 0;
index 71a9e1f..3f83e4c 100644 (file)
@@ -4003,7 +4003,7 @@ void WebViewImpl::setIsAcceleratedCompositingActive(bool active)
                 WebRect asciiToRectTable[128];
                 int fontHeight;
                 SkBitmap bitmap = WebCore::CompositorHUDFontAtlas::generateFontAtlas(asciiToRectTable, fontHeight);
-                m_layerTreeView->setFontAtlas(bitmap, asciiToRectTable, fontHeight);
+                m_layerTreeView->setFontAtlas(asciiToRectTable, bitmap, fontHeight);
             }
         } else {
             m_nonCompositedContentHost.clear();