SkImage::NewFromGenerator(SkImageGenerator*), and a unit test.

SkImage::NewFromGenerator(SkImageGenerator*), and a unit test.

Committed: https://skia.googlesource.com/skia/+/ea4673fde65f15d6f1ca77e24ced7348c4914517

[hal.canary, 2014-08-12 14:46:13]

The CQ bit was checked by halcanary@google.com

[commit-bot: I haz the power, 2014-08-12 14:46:25]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/halcanary@google.com/465823003/1

[commit-bot: I haz the power, 2014-08-12 14:46:26]

Note for Reviewers:
The CQ is waiting for an approval.
If you believe that the CL is not ready yet, or if you would like to L-G-T-M
with comments then please uncheck the CQ checkbox.

Waiting for LGTM from a valid reviewer till 2014-08-12 20:46 UTC

[commit-bot: I haz the power, 2014-08-12 14:46:29]

The CQ bit was unchecked by commit-bot@chromium.org

[hal.canary, 2014-08-12 14:46:29]

ptal

[commit-bot: I haz the power, 2014-08-12 14:46:30]

Presubmit check for 465823003-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Since the CL is editing public API, you must have an LGTM from one of:
('reed@chromium.org', 'reed@google.com', 'bsalomon@chromium.org',
'bsalomon@google.com', 'djsollen@chromium.org', 'djsollen@google.com')

[reed1, 2014-08-12 14:49:04]

lgtm

https://codereview.chromium.org/465823003/diff/1/include/core/SkImage.h
File include/core/SkImage.h (right):

https://codereview.chromium.org/465823003/diff/1/include/core/SkImage.h#newco...
include/core/SkImage.h:57: 
tiny nit: 2 lines blank?

[hal.canary, 2014-08-12 14:55:19]

https://codereview.chromium.org/465823003/diff/1/include/core/SkImage.h
File include/core/SkImage.h (right):

https://codereview.chromium.org/465823003/diff/1/include/core/SkImage.h#newco...
include/core/SkImage.h:57: 
On 2014/08/12 14:49:04, reed1 wrote:
> tiny nit: 2 lines blank?

Done.

[hal.canary, 2014-08-12 14:55:23]

The CQ bit was checked by halcanary@google.com

[commit-bot: I haz the power, 2014-08-12 14:56:30]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/halcanary@google.com/465823003/20001

[commit-bot: I haz the power, 2014-08-12 15:05:03]

Change committed as 186f7b04956a1742f185a4ca69b44b52bc50e7fc

[hal.canary, 2014-08-12 16:06:01]

A revert of this CL has been created in
https://codereview.chromium.org/453723003/ by halcanary@google.com.

The reason for reverting is: Mac bots are failing.

[hal.canary, 2014-08-14 13:56:57]

Tests pass, ready to land after M38 branch.

[hal.canary, 2014-08-18 15:12:13]

The CQ bit was checked by halcanary@google.com

[commit-bot: I haz the power, 2014-08-18 15:12:18]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/halcanary@google.com/465823003/60001

[commit-bot: I haz the power, 2014-08-18 15:27:13]

Committed patchset #3 (60001) as ea4673fde65f15d6f1ca77e24ced7348c4914517