Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(108)

Issue 160004: Restore WebFrame::GetPrintPageShrink() method originally added in rev 14639

Created:
11 years, 5 months ago by Marshall Greenblatt
Modified:
7 years, 7 months ago
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Restore the WebFrame::GetPrintPageShrink() method originally added in rev 14639 and later removed in rev 19669. From the original commit message: Rev 12100 changed the way that we spool and render printed output, most notably introducing use of the PrintContext class. The existing PrintContext::spoolPage() method applies a webkit scaling factor before rendering output to the graphics context. ChromePrintContext::spoolPage() (in webframe_impl.cc), which is used by chromium instead of PrintContext::spoolPage(), does not apply this scaling factor, but instead eventually returns the scaling factor via WebFrame::PrintPage(). This is a problem for the Chromium Embedded Framework (CEF) because, unlike chromium, the CEF renders directly to the printer device context. It is therefore important for CEF that we retrieve and apply the webkit scaling factor before calling PrintPage(). In order to support this capability the following adds a WebFrame::GetPrintPageShrink() method.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -0 lines) Patch
M webkit/glue/webframe.h View 1 chunk +5 lines, -0 lines 0 comments Download
M webkit/glue/webframe_impl.h View 1 chunk +1 line, -0 lines 0 comments Download
M webkit/glue/webframe_impl.cc View 1 chunk +10 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Marshall Greenblatt
Please review this patch. Original patch set (with Rietveld upload error): http://codereview.chromium.org/149186
11 years, 5 months ago (2009-07-23 08:53:53 UTC) #1
darin (slow to review)
On 2009/07/23 08:53:53, Marshall Greenblatt wrote: > Please review this patch. > > Original patch ...
11 years, 4 months ago (2009-08-10 06:33:38 UTC) #2
Marshall Greenblatt
11 years, 4 months ago (2009-08-13 16:37:03 UTC) #3
The updated patch set that includes the move to webkit/api is available here:

http://codereview.chromium.org/164482

Powered by Google App Engine
This is Rietveld 408576698