introduce Props to surface

src/core/SkBitmapDevice.cpp

 /*
  * Copyright 2013 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkBitmapDevice.h"
 #include "SkConfig8888.h"
 #include "SkDraw.h"
@@ skipping 39 matching lines @@
         default:
             return false;
     }
 
     if (newAlphaType) {
         *newAlphaType = canonicalAlphaType;
     }
     return true;
 }
 
-SkBitmapDevice::SkBitmapDevice(const SkBitmap& bitmap) : fBitmap(bitmap) {
+SkBitmapDevice::SkBitmapDevice(const SkBitmap& bitmap, const SkSurfaceProps* props)
+    : fBitmap(bitmap)
+    , fProps(props ? *props : SkSurfaceProps())
+{
     SkASSERT(valid_for_bitmap_device(bitmap.info(), NULL));
 }
 
 SkBitmapDevice::SkBitmapDevice(const SkBitmap& bitmap, const SkDeviceProperties& deviceProperties)
     : SkBaseDevice(deviceProperties)
     , fBitmap(bitmap)
 {
     SkASSERT(valid_for_bitmap_device(bitmap.info(), NULL));
 }
 
+SkBitmapDevice* SkBitmapDevice::Create(const SkBitmap& bitmap, const SkSurfaceProps* props) {
+    // validate bitmap?
+    return SkNEW_ARGS(SkBitmapDevice, (bitmap, props));
+}
+
 SkBitmapDevice* SkBitmapDevice::Create(const SkImageInfo& origInfo,
                                        const SkDeviceProperties* props) {
     SkAlphaType newAT = origInfo.alphaType();
     if (!valid_for_bitmap_device(origInfo, &newAT)) {
         return NULL;
     }
 
     const SkImageInfo info = origInfo.makeAlphaType(newAT);
     SkBitmap bitmap;
 
@@ skipping 286 matching lines @@
 }
 
 SkImageFilter::Cache* SkBitmapDevice::getImageFilterCache() {
     SkImageFilter::Cache* cache = SkImageFilter::Cache::Get();
     cache->ref();
     return cache;
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 
+static bool disable_lcd(const SkBitmap& target) {
+    return kN32_SkColorType != target.colorType();

[bungeman-skia, 2014/09/15 20:47:13]

comment that we do this because we don't have a blitter.

note that we do support path effect and fake bold with lcd text, this should
also include alternate fill styles.

The rasterizer and mask effect are implementation depended on why we don't do
them with lcd, it's just more code for something I doubt anyone cares that much
about.

The xfer mode I have no idea about.



I know this is in an in-between state, but the end goal is that the paint/font
has a bit on it that states 'exploit subpixel geometry' which is always passed
through to the scalar context (so that we know if to back form from lcd if
needed) and the scalar context is also told what the geometry actually is. Note
that all of these bits need to be in the scalar descriptor, because a request
for 'lcd on unknown' may look different from 'normal on unknown'.

+}
+
 bool SkBitmapDevice::filterTextFlags(const SkPaint& paint, TextFlags* flags) {
-    if (!paint.isLCDRenderText() || !paint.isAntiAlias()) {
-        // we're cool with the paint as is
-        return false;
+    uint32_t clearMask = 0;
+
+    if (paint.isLCDRenderText() && paint.isAntiAlias() && disable_lcd(fBitmap)) {
+        clearMask |= SkPaint::kLCDRenderText_Flag;
     }
 
-    if (kN32_SkColorType != fBitmap.colorType() ||
-        paint.getRasterizer() ||
-        paint.getPathEffect() ||
-        paint.isFakeBoldText() ||
-        paint.getStyle() != SkPaint::kFill_Style ||
-        !SkXfermode::IsMode(paint.getXfermode(), SkXfermode::kSrcOver_Mode)) {
-        // turn off lcd
-        flags->fFlags = paint.getFlags() & ~SkPaint::kLCDRenderText_Flag;
+    if (clearMask) {
+        flags->fFlags = paint.getFlags() & ~clearMask;
         flags->fHinting = paint.getHinting();
         return true;
     }
     // we're cool with the paint as is
     return false;
 }