Just pass the draw context to paint conversion

Just pass the draw context to paint conversion

Concentrate on using draw context functionality to answer gamma and color
related queries in more places.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2182603003

Committed: https://skia.googlesource.com/skia/+/8fe485b793f6b3c286700988138e1395b0d33487

[Brian Osman, 2016-07-25 17:28:47]

Description was changed from

==========
Just pass the draw context to paint conversion

Concentrate on using draw context functionality to answer gamma and color
related queries in more places.

BUG=skia:
==========

to

==========
Just pass the draw context to paint conversion

Concentrate on using draw context functionality to answer gamma and color
related queries in more places.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2182603003
==========

[Brian Osman, 2016-07-25 17:30:26]

The CQ bit was checked by brianosman@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-25 17:30:34]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-25 17:32:02]

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

[commit-bot: I haz the power, 2016-07-25 17:32:02]

Dry run: Try jobs failed on following builders:
  Build-Ubuntu-GCC-Arm7-Debug-Android-Trybot on master.client.skia.compile
(JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-Arm...)
  Build-Ubuntu-GCC-x86_64-Release-Trybot on master.client.skia.compile
(JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-x86...)

[Brian Osman, 2016-07-25 17:32:24]

brianosman@google.com changed reviewers:
+ bsalomon@google.com, robertphillips@google.com

[Brian Osman, 2016-07-25 17:32:25]

My plan is to eliminate the gamma-correct flag from SkSurfaceProps, to eliminate
API redundancy. This is step one. After I fix a few other dangling references,
I'll rewire GrDrawContext::isGammaCorrect() to be: SkToBool(fColorSpace.get()).

[Brian Osman, 2016-07-25 17:39:41]

The CQ bit was checked by brianosman@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-25 17:39:42]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-25 17:56:32]

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

[commit-bot: I haz the power, 2016-07-25 17:56:33]

Dry run: This issue passed the CQ dry run.

[bsalomon, 2016-07-25 19:22:17]

lgtm

[Brian Osman, 2016-07-25 19:30:35]

The CQ bit was checked by brianosman@google.com

[commit-bot: I haz the power, 2016-07-25 19:30:41]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-25 19:31:54]

Description was changed from

==========
Just pass the draw context to paint conversion

Concentrate on using draw context functionality to answer gamma and color
related queries in more places.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2182603003
==========

to

==========
Just pass the draw context to paint conversion

Concentrate on using draw context functionality to answer gamma and color
related queries in more places.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2182603003

Committed:
https://skia.googlesource.com/skia/+/8fe485b793f6b3c286700988138e1395b0d33487
==========

[commit-bot: I haz the power, 2016-07-25 19:31:54]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/8fe485b793f6b3c286700988138e1395b0d33487

[robertphillips, 2016-07-25 19:32:02]

lgtm too

https://codereview.chromium.org/2182603003/diff/20001/src/gpu/text/GrAtlasTex...
File src/gpu/text/GrAtlasTextContext.cpp (right):

https://codereview.chromium.org/2182603003/diff/20001/src/gpu/text/GrAtlasTex...
src/gpu/text/GrAtlasTextContext.cpp:391: // Setup dummy SkPaint / GrPaint /
GrDrawContext
Can this be 64x64 ?

[Brian Osman, 2016-07-25 19:44:13]

On 2016/07/25 19:32:02, robertphillips wrote:
> lgtm too
> 
>
https://codereview.chromium.org/2182603003/diff/20001/src/gpu/text/GrAtlasTex...
> File src/gpu/text/GrAtlasTextContext.cpp (right):
> 
>
https://codereview.chromium.org/2182603003/diff/20001/src/gpu/text/GrAtlasTex...
> src/gpu/text/GrAtlasTextContext.cpp:391: // Setup dummy SkPaint / GrPaint /
> GrDrawContext
> Can this be 64x64 ?

It probably could be. The test code below is generating blobs with positions
that are roughly in the
(0, 1024) range, and I didn't want to alter anything (even if it's just a matter
of early culling).
May not matter, since this seems to be largely a structural test?