mirrors/electron
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

use base::Owned instead of base::Passed for repeating handler

Browse source
This commit is contained in:
Jeremy Apthorp 2019-05-03 14:20:43 -07:00 committed by Nitish Sakhawalkar
parent 6c6555c13c
commit 6f0524d87f
1 changed files with 14 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
atom/browser/api/atom_api_session.cc
View file

@ -457,23 +457,28 @@ void Session::SetPermissionRequestHandler(v8::Local<v8::Value> val,
using RequestHandler = using RequestHandler =
base::Callback<void(content::WebContents*, content::PermissionType, base::Callback<void(content::WebContents*, content::PermissionType,
StatusCallback, const base::DictionaryValue&)>; StatusCallback, const base::DictionaryValue&)>;
RequestHandler handler; auto* permission_manager = static_cast<AtomPermissionManager*>(
if (!(val->IsNull() || mate::ConvertFromV8(args->isolate(), val, &handler))) { browser_context()->GetPermissionControllerDelegate());
if (val->IsNull()) {
permission_manager->SetPermissionRequestHandler(
AtomPermissionManager::RequestHandler());
return;
}
auto handler = std::make_unique<RequestHandler>();
if (!mate::ConvertFromV8(args->isolate(), val, handler.get())) {
args->ThrowError("Must pass null or function"); args->ThrowError("Must pass null or function");
return; return;
} }
auto* permission_manager = static_cast<AtomPermissionManager*>(
browser_context()->GetPermissionControllerDelegate());
permission_manager->SetPermissionRequestHandler(base::BindRepeating( permission_manager->SetPermissionRequestHandler(base::BindRepeating(
[](const RequestHandler& handler, content::WebContents* web_contents, [](RequestHandler* handler, content::WebContents* web_contents,
content::PermissionType permission_type, content::PermissionType permission_type,
AtomPermissionManager::StatusCallback callback, AtomPermissionManager::StatusCallback callback,
const base::DictionaryValue& details) { const base::DictionaryValue& details) {
handler.Run(web_contents, permission_type, handler->Run(web_contents, permission_type,
base::AdaptCallbackForRepeating(std::move(callback)), base::AdaptCallbackForRepeating(std::move(callback)),
details); details);
}, },
base::Passed(std::move(handler)))); base::Owned(std::move(handler))));
} }
void Session::SetPermissionCheckHandler(v8::Local<v8::Value> val, void Session::SetPermissionCheckHandler(v8::Local<v8::Value> val,