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

Unified Diff: third_party/WebKit/Source/bindings/core/v8/ArrayValue.cpp

Issue 2272313003: binding: Makes ExceptionState STACK_ALLOCATED(). (Closed)
Patch Set: Synced. Created 4 years, 4 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
Index: third_party/WebKit/Source/bindings/core/v8/ArrayValue.cpp
diff --git a/third_party/WebKit/Source/bindings/core/v8/ArrayValue.cpp b/third_party/WebKit/Source/bindings/core/v8/ArrayValue.cpp
index 0a49c568428100cf6370804c2634fb2262c581d5..1de9bb78183351b11d503e3d40542bda8888e5b9 100644
--- a/third_party/WebKit/Source/bindings/core/v8/ArrayValue.cpp
+++ b/third_party/WebKit/Source/bindings/core/v8/ArrayValue.cpp
@@ -39,7 +39,7 @@ ArrayValue& ArrayValue::operator=(const ArrayValue& other)
bool ArrayValue::isUndefinedOrNull() const
{
- return m_array.IsEmpty() || blink::isUndefinedOrNull(m_array);
+ return blink::isUndefinedOrNull(m_array);
}
bool ArrayValue::length(size_t& length) const
@@ -59,13 +59,13 @@ bool ArrayValue::get(size_t index, Dictionary& value) const
if (index >= m_array->Length())
return false;
- ASSERT(m_isolate);
- ASSERT(m_isolate == v8::Isolate::GetCurrent());
+ DCHECK(m_isolate);
+ DCHECK_EQ(m_isolate, v8::Isolate::GetCurrent());
v8::Local<v8::Value> indexedValue;
if (!m_array->Get(m_isolate->GetCurrentContext(), index).ToLocal(&indexedValue) || !indexedValue->IsObject())
return false;
- value = Dictionary(indexedValue, m_isolate, m_exceptionState);
+ value = Dictionary(m_isolate, indexedValue);
return true;
}
« no previous file with comments | « third_party/WebKit/Source/bindings/core/v8/ArrayValue.h ('k') | third_party/WebKit/Source/bindings/core/v8/Dictionary.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698