Extracted FlattenTransparency method.

Extracted FlattenTransparency method.

No functionality was changed.
Just exctracting some code. Method growing to large and I am going to extend RenderPage functionality.

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

[gene1, 2012-08-16 22:31:08]

lgtm

[Lei Zhang, 2012-08-17 02:30:00]

lgtm with nit below

https://chromiumcodereview.appspot.com/10830364/diff/1/chrome/renderer/print_...
File chrome/renderer/print_web_view_helper_win.cc (right):

https://chromiumcodereview.appspot.com/10830364/diff/1/chrome/renderer/print_...
chrome/renderer/print_web_view_helper_win.cc:96: Metafile*
FlattenTransparency(Metafile* metafile, gfx::Size page_size) {
gfx::Size -> const gfx::Size& ?

[Vitaly Buka (NO REVIEWS), 2012-08-17 02:40:32]

https://chromiumcodereview.appspot.com/10830364/diff/1/chrome/renderer/print_...
File chrome/renderer/print_web_view_helper_win.cc (right):

https://chromiumcodereview.appspot.com/10830364/diff/1/chrome/renderer/print_...
chrome/renderer/print_web_view_helper_win.cc:96: Metafile*
FlattenTransparency(Metafile* metafile, gfx::Size page_size) {
On 2012/08/17 02:30:00, Lei Zhang wrote:
> gfx::Size -> const gfx::Size& ?
Done. But it's just 64bit. It's below my threshold :-)

[commit-bot: I haz the power, 2012-08-17 02:40:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/vitalybuka@chromium.org/10830364/6001

[commit-bot: I haz the power, 2012-08-17 05:25:23]

Change committed as 152054