Fixed return value in PrintingContext::OnError().

printing/printing_context.cc

 // Copyright (c) 2011 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 "printing/printing_context.h"
 
 #include "base/logging.h"
 #include "base/values.h"
 #include "printing/page_setup.h"
 #include "printing/page_size_margins.h"
@@ skipping 26 matching lines @@
   ReleaseContext();
 
   settings_.Clear();
 
   in_print_job_ = false;
   dialog_box_dismissed_ = false;
   abort_printing_ = false;
 }
 
 PrintingContext::Result PrintingContext::OnError() {
+  Result result = abort_printing_ ? CANCEL : FAILED;
   ResetSettings();
-  return abort_printing_ ? CANCEL : FAILED;
+  return result;
 }
 
 PrintingContext::Result PrintingContext::UsePdfSettings() {
   scoped_ptr<base::DictionaryValue> pdf_settings(new base::DictionaryValue);
   pdf_settings->SetBoolean(kSettingHeaderFooterEnabled, false);
   pdf_settings->SetBoolean(kSettingShouldPrintBackgrounds, false);
   pdf_settings->SetBoolean(kSettingShouldPrintSelectionOnly, false);
   pdf_settings->SetInteger(kSettingMarginsType, printing::NO_MARGINS);
   pdf_settings->SetBoolean(kSettingCollate, true);
   pdf_settings->SetInteger(kSettingCopies, 1);
@@ skipping 50 matching lines @@
           kCloudPrintMarginInch * settings_.device_units_per_inch());
     }
     settings_.SetPrinterPrintableArea(paper_size, paper_rect, true);
     return OK;
   }
 
   return UpdatePrinterSettings(open_in_external_preview);
 }
 
 }  // namespace printing