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

Allow passing null to webRequest

Browse source
This commit is contained in:
Cheng Zhao 2015-12-11 19:11:09 +08:00
parent b7fce578d5
commit 62f4b25cf9
3 changed files with 6 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
atom/browser/api/atom_api_web_request.cc
View file

@ -28,13 +28,14 @@ WebRequest::~WebRequest() {
template<AtomNetworkDelegate::EventTypes type>
void WebRequest::SetListener(mate::Arguments* args) {
DCHECK_CURRENTLY_ON(BrowserThread::UI);
scoped_ptr<base::DictionaryValue> filter(new base::DictionaryValue());
scoped_ptr<base::DictionaryValue> filter(new base::DictionaryValue);
args->GetNext(filter.get());
v8::Local<v8::Value> value;
AtomNetworkDelegate::Listener callback;
if (!args->GetNext(&callback)) {
args->ThrowError("Must pass null or a function");
if (!args->GetNext(&callback) &&
!(args->GetNext(&value) && value->IsNull())) {
args->ThrowError("Must pass null or a Function");
return;
}

2
atom/browser/api/atom_api_web_request.h
View file

@ -5,8 +5,6 @@
#ifndef ATOM_BROWSER_API_ATOM_API_WEB_REQUEST_H_
#define ATOM_BROWSER_API_ATOM_API_WEB_REQUEST_H_
#include <string>
#include "atom/browser/api/trackable_object.h"
#include "atom/browser/net/atom_network_delegate.h"
#include "native_mate/arguments.h"

1
atom/browser/net/atom_network_delegate.h
View file

@ -7,7 +7,6 @@
#include <map>
#include <set>
#include <string>
#include "brightray/browser/network_delegate.h"
#include "base/callback.h"