binding: Makes ExceptionState STACK_ALLOCATED().

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

 /*
  * Copyright (C) 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 21 matching lines @@
 
 ArrayValue& ArrayValue::operator=(const ArrayValue& other)
 {
     m_array = other.m_array;
     m_isolate = other.m_isolate;
     return *this;
 }
 
 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
 {
     if (isUndefinedOrNull())
         return false;
 
     length = m_array->Length();
     return true;
 }
 
 bool ArrayValue::get(size_t index, Dictionary& value) const
 {
     if (isUndefinedOrNull())
         return false;
 
     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;
 }
 
 } // namespace blink