binding: Makes ExceptionState STACK_ALLOCATED().

third_party/WebKit/Source/bindings/core/v8/ArrayValue.cpp

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);

[peria, 2016/08/25 15:31:18]

We don't need 'blink::' here.

[Yuki, 2016/08/30 06:09:07]

We do need it because ArrayValue::isUndefinedOrNull shadows
blink::isUndefinedOrNull.

 }
 
 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;
 }