Change return type of PrintPage

content/renderer/pepper/pepper_webplugin_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/pepper/pepper_webplugin_impl.h"
 
 #include <cmath>
 
 #include "base/debug/crash_logging.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 257 matching lines @@
 
 int PepperWebPluginImpl::printBegin(const WebPrintParams& print_params) {
   return instance_->PrintBegin(print_params);
 }
 
 void PepperWebPluginImpl::printPage(int page_number, blink::WebCanvas* canvas,
                                     bool unused) {
   return instance_->PrintPage(page_number, canvas);
 }
 
-bool PepperWebPluginImpl::printPage(int page_number, blink::WebCanvas* canvas) {
-  instance_->PrintPage(page_number, canvas);
-  return true;
+void PepperWebPluginImpl::printPage(int page_number, blink::WebCanvas* canvas) {
+  return instance_->PrintPage(page_number, canvas);

[raymes, 2015/06/12 00:01:52]

This "return" can be removed right?

[Habib Virji, 2015/06/12 13:29:18]

That's true removed. Also below printEnd which does same has also been removed. 

 }
 
+

[piman, 2015/06/11 22:45:14]

nit: please remove redundant empty line

[Habib Virji, 2015/06/12 13:29:18]

Thanks. Done. 

 void PepperWebPluginImpl::printEnd() { return instance_->PrintEnd(); }
 
 bool PepperWebPluginImpl::getPrintPresetOptionsFromDocument(
     blink::WebPrintPresetOptions* preset_options) {
   return instance_->GetPrintPresetOptionsFromDocument(preset_options);
 }
 
 bool PepperWebPluginImpl::canRotateView() { return instance_->CanRotateView(); }
 
 void PepperWebPluginImpl::rotateView(RotationType type) {
   instance_->RotateView(type);
 }
 
 bool PepperWebPluginImpl::isPlaceholder() { return false; }
 
 }  // namespace content