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

Side by Side Diff: chrome/browser/printing/print_dialog_cloud.cc

Issue 7550022: Print Preview: Fixing behavior of event listeners. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressing comments Created 9 years, 4 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | chrome/browser/printing/print_preview_message_handler.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/printing/print_dialog_cloud.h" 5 #include "chrome/browser/printing/print_dialog_cloud.h"
6 #include "chrome/browser/printing/print_dialog_cloud_internal.h" 6 #include "chrome/browser/printing/print_dialog_cloud_internal.h"
7 7
8 #include "base/base64.h" 8 #include "base/base64.h"
9 #include "base/command_line.h" 9 #include "base/command_line.h"
10 #include "base/file_util.h" 10 #include "base/file_util.h"
(...skipping 359 matching lines...) Expand 10 before | Expand all | Expand 10 after
370 const double kMaxPageShrink = 2.0; 370 const double kMaxPageShrink = 2.0;
371 371
372 PrintMsg_Print_Params default_settings; 372 PrintMsg_Print_Params default_settings;
373 default_settings.printable_size = gfx::Size(kWidth, kHeight); 373 default_settings.printable_size = gfx::Size(kWidth, kHeight);
374 default_settings.dpi = kDPI; 374 default_settings.dpi = kDPI;
375 default_settings.min_shrink = kMinPageShrink; 375 default_settings.min_shrink = kMinPageShrink;
376 default_settings.max_shrink = kMaxPageShrink; 376 default_settings.max_shrink = kMaxPageShrink;
377 default_settings.desired_dpi = kDPI; 377 default_settings.desired_dpi = kDPI;
378 default_settings.document_cookie = 0; 378 default_settings.document_cookie = 0;
379 default_settings.selection_only = false; 379 default_settings.selection_only = false;
380 default_settings.preview_request_id = 0;
381 default_settings.is_first_request = true;
380 382
381 if (!GetPageSetupParameters(json, default_settings)) { 383 if (!GetPageSetupParameters(json, default_settings)) {
382 NOTREACHED(); 384 NOTREACHED();
383 return; 385 return;
384 } 386 }
385 387
386 // TODO(scottbyer) - Here is where we would kick the originating 388 // TODO(scottbyer) - Here is where we would kick the originating
387 // renderer thread with these new parameters in order to get it to 389 // renderer thread with these new parameters in order to get it to
388 // re-generate the PDF data and hand it back to us. window.print() is 390 // re-generate the PDF data and hand it back to us. window.print() is
389 // currently synchronous, so there's a lot of work to do to get to 391 // currently synchronous, so there's a lot of work to do to get to
(...skipping 243 matching lines...) Expand 10 before | Expand all | Expand 10 after
633 print_job_title, 635 print_job_title,
634 file_type, 636 file_type,
635 false); 637 false);
636 return true; 638 return true;
637 } 639 }
638 } 640 }
639 return false; 641 return false;
640 } 642 }
641 643
642 } // end namespace 644 } // end namespace
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/printing/print_preview_message_handler.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698