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

Unified Diff: extensions/renderer/api_bindings_system_unittest.cc

Issue 2575173002: [Extensions Bindings] Add a bridge to use current custom bindings (Closed)
Patch Set: . Created 4 years 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/api_bindings_system.cc ('k') | extensions/renderer/api_request_handler.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/renderer/api_bindings_system_unittest.cc
diff --git a/extensions/renderer/api_bindings_system_unittest.cc b/extensions/renderer/api_bindings_system_unittest.cc
index 654f11a516a29f50b64a62025dcd250685e6b7b8..5a4ec2259d9f2b5f4f045ebc62d9b5f618a30250 100644
--- a/extensions/renderer/api_bindings_system_unittest.cc
+++ b/extensions/renderer/api_bindings_system_unittest.cc
@@ -210,10 +210,10 @@ TEST_F(APIBindingsSystemTest, TestInitializationAndCallbacks) {
v8::Local<v8::Context> context = ContextLocal();
v8::Local<v8::Object> alpha_api = bindings_system()->CreateAPIInstance(
- kAlphaAPIName, context, isolate(), base::Bind(&AllowAllAPIs));
+ kAlphaAPIName, context, isolate(), base::Bind(&AllowAllAPIs), nullptr);
ASSERT_FALSE(alpha_api.IsEmpty());
v8::Local<v8::Object> beta_api = bindings_system()->CreateAPIInstance(
- kBetaAPIName, context, isolate(), base::Bind(&AllowAllAPIs));
+ kBetaAPIName, context, isolate(), base::Bind(&AllowAllAPIs), nullptr);
ASSERT_FALSE(beta_api.IsEmpty());
{
@@ -321,7 +321,7 @@ TEST_F(APIBindingsSystemTest, TestCustomHooks) {
base::Bind(hook, &did_call));
v8::Local<v8::Object> alpha_api = bindings_system()->CreateAPIInstance(
- kAlphaAPIName, context, isolate(), base::Bind(&AllowAllAPIs));
+ kAlphaAPIName, context, isolate(), base::Bind(&AllowAllAPIs), nullptr);
ASSERT_FALSE(alpha_api.IsEmpty());
{
@@ -408,7 +408,8 @@ TEST_F(APIBindingsSystemTestWithRealAPI, RealAPIs) {
auto add_api_to_chrome = [this, &chrome,
&context](const std::string& api_name) {
v8::Local<v8::Object> api = bindings_system()->CreateAPIInstance(
- api_name, context, context->GetIsolate(), base::Bind(&AllowAllAPIs));
+ api_name, context, context->GetIsolate(), base::Bind(&AllowAllAPIs),
+ nullptr);
ASSERT_FALSE(api.IsEmpty()) << api_name;
v8::Maybe<bool> res = chrome->Set(
context, gin::StringToV8(context->GetIsolate(), api_name), api);
« no previous file with comments | « extensions/renderer/api_bindings_system.cc ('k') | extensions/renderer/api_request_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698