Fix gpu memory accounting for auto-resolving MSAA.

Fix gpu memory accounting for auto-resolving MSAA.

BUG=skia:

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

[Stephen White, 2015-04-30 17:01:01]

senorblanco@chromium.org changed reviewers:
+ bsalomon@google.com

[Stephen White, 2015-04-30 17:01:15]

Brian: PTAL

[Stephen White, 2015-04-30 17:05:27]

The CQ bit was checked by senorblanco@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-04-30 17:05:37]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1115233002/20001

[bsalomon, 2015-04-30 17:34:28]

https://codereview.chromium.org/1115233002/diff/20001/src/gpu/gl/GrGLRenderTa...
File src/gpu/gl/GrGLRenderTarget.cpp (right):

https://codereview.chromium.org/1115233002/diff/20001/src/gpu/gl/GrGLRenderTa...
src/gpu/gl/GrGLRenderTarget.cpp:48: fColorValuesPerPixel = 1;
looks ok to me, but can we rename this to something like
fColorSamplesForSizeComputation or something that makes it more clear that this
is really about byte size computation and not logical sample count? Or, we could
just compute the total size here and store that instead of the sample cnt. I
think it didn't work that way in the past because we didn't yet know whether we
had a stencil buffer. However, this class no longer has to account for the
stencil's memory

[Stephen White, 2015-04-30 17:36:18]

On 2015/04/30 17:34:28, bsalomon wrote:
>
https://codereview.chromium.org/1115233002/diff/20001/src/gpu/gl/GrGLRenderTa...
> File src/gpu/gl/GrGLRenderTarget.cpp (right):
> 
>
https://codereview.chromium.org/1115233002/diff/20001/src/gpu/gl/GrGLRenderTa...
> src/gpu/gl/GrGLRenderTarget.cpp:48: fColorValuesPerPixel = 1;
> looks ok to me, but can we rename this to something like
> fColorSamplesForSizeComputation or something that makes it more clear that
this
> is really about byte size computation and not logical sample count? Or, we
could
> just compute the total size here and store that instead of the sample cnt. I
> think it didn't work that way in the past because we didn't yet know whether
we
> had a stencil buffer. However, this class no longer has to account for the
> stencil's memory

Sure, will do.

[Stephen White, 2015-04-30 17:41:23]

The CQ bit was checked by senorblanco@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-04-30 17:41:45]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1115233002/40001

[Stephen White, 2015-04-30 17:45:14]

New patch up; PTAL. Thanks!

[bsalomon, 2015-04-30 18:45:06]

lgtm

[Stephen White, 2015-04-30 18:53:45]

The CQ bit was unchecked by senorblanco@chromium.org

[Stephen White, 2015-04-30 18:53:46]

The CQ bit was checked by senorblanco@chromium.org

[commit-bot: I haz the power, 2015-04-30 18:53:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1115233002/40001

[commit-bot: I haz the power, 2015-04-30 19:06:16]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/d298121f5d753b21c62709ae33ac08ddb053c719