Workaround Skia clipping bug in print_web_view_helper.cc.

Workaround Skia clipping bug in print_web_view_helper.cc.

BUG=133548
TEST=Test that text isn't clipped incorrectly when enabling RenderTextMac code in print_web_view_helper.cc 

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=144795

[kmadhusu, 2012-06-28 17:56:31]

Since you had initial discussions about the workaround with vandebo@, I am
deferring this review to him.

Thanks.

[vandebo (ex-Chrome), 2012-06-28 18:07:27]

LGTM w/ nit

http://codereview.chromium.org/10703027/diff/1/chrome/renderer/print_web_view...
File chrome/renderer/print_web_view_helper.cc (right):

http://codereview.chromium.org/10703027/diff/1/chrome/renderer/print_web_view...
chrome/renderer/print_web_view_helper.cc:432: //                  Remove it when
the underlying Skia bug has been fixed.
nit: "Remove" should align with "TODO"

[Alexei Svitkine (slow), 2012-06-28 18:29:18]

http://codereview.chromium.org/10703027/diff/1/chrome/renderer/print_web_view...
File chrome/renderer/print_web_view_helper.cc (right):

http://codereview.chromium.org/10703027/diff/1/chrome/renderer/print_web_view...
chrome/renderer/print_web_view_helper.cc:432: //                  Remove it when
the underlying Skia bug has been fixed.
On 2012/06/28 18:07:27, vandebo wrote:
> nit: "Remove" should align with "TODO"

Done.

[commit-bot: I haz the power, 2012-06-28 18:29:24]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/asvitkine@chromium.org/10703027/1002

[commit-bot: I haz the power, 2012-06-28 21:11:06]

Change committed as 144795