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

Unified Diff: chrome/renderer/print_web_view_helper_linux.cc

Issue 200122: Remove code doing a no-op due to float -> int rounding. (Closed)
Patch Set: Created 11 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/gtk/tabs/tab_gtk.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/print_web_view_helper_linux.cc
diff --git a/chrome/renderer/print_web_view_helper_linux.cc b/chrome/renderer/print_web_view_helper_linux.cc
index b16b05ecaa978f3b3963581aabf10f5b9b91447c..211eb6f04a9c045123be4d31d6620965e32ef797 100644
--- a/chrome/renderer/print_web_view_helper_linux.cc
+++ b/chrome/renderer/print_web_view_helper_linux.cc
@@ -27,15 +27,15 @@ void PrintWebViewHelper::Print(WebFrame* frame, bool script_initiated) {
// Top = 0.25 in.
// Bottom = 0.56 in.
const int kDPI = 72;
- const int kWidth = (8.5-0.25-0.25) * kDPI;
- const int kHeight = (11-0.25-0.56) * kDPI;
+ const int kWidth = static_cast<int>((8.5-0.25-0.25) * kDPI);
+ const int kHeight = static_cast<int>((11-0.25-0.56) * kDPI);
ViewMsg_Print_Params default_settings;
default_settings.printable_size = gfx::Size(kWidth, kHeight);
default_settings.dpi = kDPI;
default_settings.min_shrink = 1.25;
default_settings.max_shrink = 2.0;
default_settings.desired_dpi = kDPI;
- default_settings.document_cookie = NULL;
+ default_settings.document_cookie = 0;
default_settings.selection_only = false;
ViewMsg_PrintPages_Params print_settings;
« no previous file with comments | « chrome/browser/gtk/tabs/tab_gtk.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698