diff options
| author | maruel@google.com <maruel@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-08-14 21:16:24 +0000 |
|---|---|---|
| committer | maruel@google.com <maruel@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-08-14 21:16:24 +0000 |
| commit | 8ecd00f29e3edf1cccd1d9cdb9d82984f20b5dcd (patch) | |
| tree | 09a94ac39d26132e2821018af728f13744efddec /chrome/browser | |
| parent | 9b7e4497c3862ec47a97a3f726b33b4c8ed401b4 (diff) | |
| download | chromium_src-8ecd00f29e3edf1cccd1d9cdb9d82984f20b5dcd.zip chromium_src-8ecd00f29e3edf1cccd1d9cdb9d82984f20b5dcd.tar.gz chromium_src-8ecd00f29e3edf1cccd1d9cdb9d82984f20b5dcd.tar.bz2 | |
Some printers advertise longer printable length than the page length. Remove the DCHECK that was verifying this. I would have liked to had a buffer to accept ~5 inch of difference but at that point, the dpi is not known.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@893 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser')
| -rw-r--r-- | chrome/browser/printing/page_setup.cc | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/chrome/browser/printing/page_setup.cc b/chrome/browser/printing/page_setup.cc index 6d6c42a..66d904a 100644 --- a/chrome/browser/printing/page_setup.cc +++ b/chrome/browser/printing/page_setup.cc @@ -86,9 +86,9 @@ void PageSetup::Init(const gfx::Size& physical_size, const gfx::Rect& printable_area, int text_height) { DCHECK_LE(printable_area.right(), physical_size.width()); - // I've seen this assert triggers on Canon GP160PF PCL 5e. - // 28092 vs. 27940 @ 600 dpi = ~.25 inch. - DCHECK_LE(printable_area.bottom(), physical_size.height()); + // I've seen this assert triggers on Canon GP160PF PCL 5e and HP LaserJet 5. + // Since we don't know the dpi here, just disable the check. + // DCHECK_LE(printable_area.bottom(), physical_size.height()); DCHECK_GE(printable_area.x(), 0); DCHECK_GE(printable_area.y(), 0); DCHECK_GE(text_height, 0); |