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

Use ToDetailString for processing error message

Browse source
This commit is contained in:
Kevin Sawicki 2016-08-10 09:34:33 -07:00
parent fa483e1fe0
commit 1dbaec0edd
1 changed files with 5 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

15
native_mate/arguments.cc
View file

@ -11,16 +11,11 @@ namespace mate {
namespace { namespace {
std::string V8TypeAsString(v8::Local<v8::Value> value) { std::string V8TypeAsString(v8::Isolate* isolate, v8::Local<v8::Value> value) {
if (value.IsEmpty()) v8::MaybeLocal<v8::String> details = value->ToDetailString(isolate);
return "<empty handle>";
if (value->IsUndefined())
return "undefined";
if (value->IsNull())
return "null";
std::string result; std::string result;
if (!ConvertFromV8(NULL, value, &result)) if (!details.IsEmpty())
return std::string(); ConvertFromV8(isolate, details.ToLocalChecked(), &result);
return result; return result;
} }
@ -55,7 +50,7 @@ v8::Local<v8::Value> Arguments::ThrowError() const {
return ThrowTypeError(base::StringPrintf( return ThrowTypeError(base::StringPrintf(
"Error processing argument at index %d, conversion failure from %s", "Error processing argument at index %d, conversion failure from %s",
next_, V8TypeAsString((*info_)[next_]).c_str())); next_, V8TypeAsString(isolate_, (*info_)[next_]).c_str()));
} }
v8::Local<v8::Value> Arguments::ThrowError(const std::string& message) const { v8::Local<v8::Value> Arguments::ThrowError(const std::string& message) const {