Initialize bounds even if the text length is zero.

Initialize bounds even if the text length is zero.

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

[reed1, 2013-03-04 19:11:47]

code looks good, question about the test.

https://codereview.chromium.org/12387092/diff/1/tests/PaintTest.cpp
File tests/PaintTest.cpp (right):

https://codereview.chromium.org/12387092/diff/1/tests/PaintTest.cpp#newcode108
tests/PaintTest.cpp:108: // test that the rect was initialized
Might get lucky if 'r' is already empty. Should we explicitly set it to some
non-empty values first?

[djsollen, 2013-03-04 19:47:04]

https://codereview.chromium.org/12387092/diff/1/tests/PaintTest.cpp
File tests/PaintTest.cpp (right):

https://codereview.chromium.org/12387092/diff/1/tests/PaintTest.cpp#newcode108
tests/PaintTest.cpp:108: // test that the rect was initialized
On 2013/03/04 19:11:47, reed1 wrote:
> Might get lucky if 'r' is already empty. Should we explicitly set it to some
> non-empty values first?

Done.

[commit-bot: I haz the power, 2013-03-04 19:47:17]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[djsollen, 2013-03-04 19:47:47]

Committed patchset #2 manually as r7964 (presubmit successful).