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

Unified Diff: printing/printing_context_mac.mm

Issue 1233983004: Remove base/mac/scoped_nsexception_enabler.h. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 5 years, 5 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
Index: printing/printing_context_mac.mm
diff --git a/printing/printing_context_mac.mm b/printing/printing_context_mac.mm
index 8689897be4e4cac716f96e0b02b82c3c4365dd8b..36badc37d55504fbe13b7c283903007db366a195 100644
--- a/printing/printing_context_mac.mm
+++ b/printing/printing_context_mac.mm
@@ -12,7 +12,6 @@
#include "base/logging.h"
#include "base/mac/scoped_cftyperef.h"
#include "base/mac/scoped_nsautorelease_pool.h"
-#include "base/mac/scoped_nsexception_enabler.h"
#include "base/strings/sys_string_conversions.h"
#include "base/strings/utf_string_conversions.h"
#include "base/values.h"
@@ -86,11 +85,6 @@ void PrintingContextMac::AskUserForSettings(
bool has_selection,
bool is_scripted,
const PrintSettingsCallback& callback) {
- // Third-party print drivers seem to be an area prone to raising exceptions.
- // This will allow exceptions to be raised, but does not handle them. The
- // NSPrintPanel appears to have appropriate NSException handlers.
- base::mac::ScopedNSExceptionEnabler enabler;
-
// Exceptions can also happen when the NSPrintPanel is being
// deallocated, so it must be autoreleased within this scope.
base::mac::ScopedNSAutoreleasePool pool;

Powered by Google App Engine
This is Rietveld 408576698