Start actually bounding some draw ops.

Start actually bounding some draw ops.

This covers most of the common draws.

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/62b67ae96e94fd22569b058a3bc4625b9f52ed56

[mtklein, 2014-08-18 17:13:12]

The CQ bit was checked by mtklein@google.com

[commit-bot: I haz the power, 2014-08-18 17:13:24]

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/mtklein@chromium.org/469213007/60001

[commit-bot: I haz the power, 2014-08-18 17:13:25]

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-18 23:13 UTC

[commit-bot: I haz the power, 2014-08-18 17:20:43]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  Test-Ubuntu13.10-GCE-NoGPU-x86_64-Debug-Trybot on tryserver.skia
(http://108.170.220.120:10117/builders/Test-Ubuntu13.10-GCE-NoGPU-x86_64-Debug...)
  Test-Ubuntu13.10-GCE-NoGPU-x86_64-Release-Shared-Trybot on tryserver.skia
(http://108.170.220.120:10117/builders/Test-Ubuntu13.10-GCE-NoGPU-x86_64-Relea...)

[commit-bot: I haz the power, 2014-08-18 17:24:03]

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

[commit-bot: I haz the power, 2014-08-18 17:24:05]

Try jobs failed on following builders:
  Test-Ubuntu13.10-GCE-NoGPU-x86_64-Release-Shared-Trybot on tryserver.skia
(http://108.170.220.120:10117/builders/Test-Ubuntu13.10-GCE-NoGPU-x86_64-Relea...)

[robertphillips, 2014-08-18 17:48:40]

lgtm + 1 nit & 1 potential fix

https://codereview.chromium.org/469213007/diff/60001/src/core/SkRecordDraw.cpp
File src/core/SkRecordDraw.cpp (right):

https://codereview.chromium.org/469213007/diff/60001/src/core/SkRecordDraw.cp...
src/core/SkRecordDraw.cpp:289: SkRect dst;
If there are no chars we probably want an empty bbox rather than one bloated by
the font metrics.

https://codereview.chromium.org/469213007/diff/60001/src/core/SkRecordDraw.cp...
src/core/SkRecordDraw.cpp:311: // FIXME: These bounds should be tight (and
correct), but reading SkFontMetrics is likely
Double e.g.'s

[mtklein, 2014-08-18 18:01:41]

On 2014/08/18 17:48:40, robertphillips wrote:
> lgtm + 1 nit & 1 potential fix
> 
> https://codereview.chromium.org/469213007/diff/60001/src/core/SkRecordDraw.cpp
> File src/core/SkRecordDraw.cpp (right):
> 
>
https://codereview.chromium.org/469213007/diff/60001/src/core/SkRecordDraw.cp...
> src/core/SkRecordDraw.cpp:289: SkRect dst;
> If there are no chars we probably want an empty bbox rather than one bloated
by
> the font metrics.
> 
>
https://codereview.chromium.org/469213007/diff/60001/src/core/SkRecordDraw.cp...
> src/core/SkRecordDraw.cpp:311: // FIXME: These bounds should be tight (and
> correct), but reading SkFontMetrics is likely
> Double e.g.'s

Both done.

[mtklein, 2014-08-18 18:02:11]

The CQ bit was checked by mtklein@google.com

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

CQ is trying da patch. Follow status at
 https://skia-tree-status.appspot.com/cq/mtklein@chromium.org/469213007/120001

[commit-bot: I haz the power, 2014-08-18 18:10:50]

Committed patchset #7 (120001) as 62b67ae96e94fd22569b058a3bc4625b9f52ed56