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

Unified Diff: extensions/renderer/bindings/api_bindings_system.cc

Issue 2961103002: [Extensions Bindings] Add an ExceptionHandler class (Closed)
Patch Set: rebase + nits Created 3 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: extensions/renderer/bindings/api_bindings_system.cc
diff --git a/extensions/renderer/bindings/api_bindings_system.cc b/extensions/renderer/bindings/api_bindings_system.cc
index e20e1a02070ea7d80d5d750c267a708304c84918..be402e479a6fb1639acffa176b4f398af3fbd099 100644
--- a/extensions/renderer/bindings/api_bindings_system.cc
+++ b/extensions/renderer/bindings/api_bindings_system.cc
@@ -19,10 +19,15 @@ APIBindingsSystem::APIBindingsSystem(
const APIRequestHandler::SendRequestMethod& send_request,
const APIEventHandler::EventListenersChangedMethod& event_listeners_changed,
const APIBinding::OnSilentRequest& on_silent_request,
+ const binding::AddConsoleError& add_console_error,
APILastError last_error)
: type_reference_map_(base::Bind(&APIBindingsSystem::InitializeType,
base::Unretained(this))),
- request_handler_(send_request, call_js, std::move(last_error)),
+ exception_handler_(add_console_error, call_js),
+ request_handler_(send_request,
+ call_js,
+ std::move(last_error),
+ &exception_handler_),
event_handler_(call_js, call_js_sync, event_listeners_changed),
access_checker_(is_available),
call_js_(call_js),
« no previous file with comments | « extensions/renderer/bindings/api_bindings_system.h ('k') | extensions/renderer/bindings/api_bindings_system_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698