Adding setSurface public API method to SkDeferredCanvas

Adding setSurface public API method to SkDeferredCanvas

The purpose of this change is to provide an API that Blink 2D canvas layers can use
to install a new render target when recovering from a lost graphics context.

Committed: https://code.google.com/p/skia/source/detail?r=9276

[Justin Novosad, 2013-05-24 14:33:23]

PTAL

[reed1, 2013-05-24 14:54:48]

I like the idea, but its starting to make me wonder if we should be write a
DeferredSurface class instead of Canvas.

Somewhat related, this new call relies (again) on setDevice, which we really
want to remove at some point. Can you imaging a future version of Deferred-ness
where we don't need that call?

[Justin Novosad, 2013-05-24 15:20:02]

About the DeferredSurface idea: I'm not sure I understand what you are
suggesting.

Removing setDevice is possible. I propose doing that in a separate CL. The
approach would be to use a factory method to create instances of
SkDeferredCanvas.  This would allow us to instantiate the Device first, and pass
it to the constructor, and therefore avoid calls to setDevice. With that
mechanism in place, it will be safe to assume that SkDeferredCanvas always has a
DeferredDevice installed, so we'll be able to remove all calls to setDevice
throughout SkDeferredCanvas.cpp

[Justin Novosad, 2013-05-24 15:21:16]

New Patch: added unit test

[Justin Novosad, 2013-05-24 16:08:17]

On 2013/05/24 15:21:16, junov wrote:
> New Patch: added unit test

lgty?

[reed1, 2013-05-24 17:12:15]

sorry, back from lunch.

lgtm

[Justin Novosad, 2013-05-24 17:13:04]

Committed patchset #4 manually as r9276 (presubmit successful).