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

Unified Diff: extensions/renderer/bindings/api_request_handler.h

Issue 2961103002: [Extensions Bindings] Add an ExceptionHandler class (Closed)
Patch Set: jbroman's 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_request_handler.h
diff --git a/extensions/renderer/bindings/api_request_handler.h b/extensions/renderer/bindings/api_request_handler.h
index c395aa7a605be475b9b961087e916d8b5dc3df76..326b29fed610e7b091b1a9eb5838269dc65c4843 100644
--- a/extensions/renderer/bindings/api_request_handler.h
+++ b/extensions/renderer/bindings/api_request_handler.h
@@ -21,6 +21,7 @@ class ListValue;
}
namespace extensions {
+class ExceptionHandler;
// A wrapper around a map for extension API calls. Contains all pending requests
// and the associated context and callback. Designed to be used on a single
@@ -54,7 +55,8 @@ class APIRequestHandler {
APIRequestHandler(const SendRequestMethod& send_request,
const CallJSFunction& call_js,
- APILastError last_error);
+ APILastError last_error,
+ ExceptionHandler* exception_handler);
~APIRequestHandler();
// Begins the process of processing the request. Returns the identifier of the
@@ -121,6 +123,8 @@ class APIRequestHandler {
APILastError last_error_;
+ ExceptionHandler* const exception_handler_;
lazyboy 2017/07/10 23:12:08 Comment about ownership
Devlin 2017/07/11 17:33:35 Done.
+
DISALLOW_COPY_AND_ASSIGN(APIRequestHandler);
};

Powered by Google App Engine
This is Rietveld 408576698