Mutexes in pixelrefs were done very sloppily initially. The code (a) assumes all
authorreed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81>
Thu, 17 May 2012 13:14:52 +0000 (13:14 +0000)
committerreed@google.com <reed@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81>
Thu, 17 May 2012 13:14:52 +0000 (13:14 +0000)
commitff0da4ff483ba7b4468b862949ffb3de505cba14
treeb02ce2447d3a112e7c66c52be6d80e2ece7f70ef
parentfa66294c7705831808ce7772d4328fc626d45034
Mutexes in pixelrefs were done very sloppily initially. The code (a) assumes all
pixelref subclasses want a mutex to guard their lock/unlock virtuals, and (b)
most subclasses use the same mutex for *all* of their instances, even when there
is no explicit need to guard modifying one instances with another.

When we try drawing bitmaps from multiple threads, we are seeing a lot of slow-
down from these mutexes. This CL has two changes to try to speed things up.

1. Add setPreLocked(), for pixelrefs who never need the onLockPixels
virtual to be called. This speeds up those subclasses in multithreaded environs
as it avoids the mutex lock all together (e.g. SkMallocPixelRef).

2. Add setMutex() to allow a subclass to change the mutex choice. ashmem wants
this, since its unflattening constructor cannot pass down the null, it needs
to cleanup afterwards.
Review URL: https://codereview.appspot.com/6199075

git-svn-id: http://skia.googlecode.com/svn/trunk@3985 2bbb7eff-a529-9590-31e7-b0007b416f81
include/core/SkPixelRef.h
include/core/SkThread_platform.h
src/core/SkBitmap.cpp
src/core/SkBitmapProcShader.cpp
src/core/SkMallocPixelRef.cpp
src/core/SkPixelRef.cpp
src/images/SkImageRefPool.cpp
src/ports/SkImageRef_ashmem.cpp