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

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

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
« no previous file with comments | « extensions/renderer/bindings/api_last_error.cc ('k') | extensions/renderer/bindings/api_request_handler.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 a2ce5da8ed61b090a1fd1dbfe1d76725c75b6c0b..767f9cdb81bf5b296575f72dca32f50d0e43c545 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
@@ -127,6 +129,10 @@ class APIRequestHandler {
APILastError last_error_;
+ // The exception handler for the bindings system; guaranteed to be valid
+ // during this object's lifetime.
+ ExceptionHandler* const exception_handler_;
+
DISALLOW_COPY_AND_ASSIGN(APIRequestHandler);
};
« no previous file with comments | « extensions/renderer/bindings/api_last_error.cc ('k') | extensions/renderer/bindings/api_request_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698