ARM Skia NEON patches - 16/17 - Blitmask

src/core/SkBlitMask_D32.cpp

 #include "SkBlitMask.h"
 #include "SkColor.h"
 #include "SkColorPriv.h"
 
 static void D32_A8_Color(void* SK_RESTRICT dst, size_t dstRB,
                          const void* SK_RESTRICT maskPtr, size_t maskRB,
                          SkColor color, int width, int height) {
     SkPMColor pmc = SkPreMultiplyColor(color);
     size_t dstOffset = dstRB - (width << 2);
     size_t maskOffset = maskRB - width;
@@ skipping 198 matching lines @@
 
     do {
         blit_lcd32_opaque_row(dstRow, srcRow, color, width);
         dstRow = (SkPMColor*)((char*)dstRow + dstRB);
         srcRow = (const SkPMColor*)((const char*)srcRow + maskRB);
     } while (--height != 0);
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 
-static SkBlitMask::ColorProc D32_A8_Factory(SkColor color) {
+SkBlitMask::ColorProc D32_A8_Factory(SkColor color) {

[mtklein, 2013/11/11 17:17:04]

Having to make this change makes me concerned that we don't have the right API.

[kevin.petit.not.used.account, 2013/11/11 18:10:34]

Fixed. This wasn't necessary.

     if (SK_ColorBLACK == color) {
         return D32_A8_Black;
     } else if (0xFF == SkColorGetA(color)) {
         return D32_A8_Opaque;
     } else {
         return D32_A8_Color;
     }
 }
 
 static SkBlitMask::ColorProc D32_LCD32_Factory(SkColor color) {
@@ skipping 353 matching lines @@
             if (flags & kSrcIsOpaque_RowFlag) {
                 index |= 1;
             }
             SkASSERT((size_t)index < SK_ARRAY_COUNT(gProcs));
             return gProcs[index];
         default:
             break;
     }
     return NULL;
 }