Rename DOM.getLayoutTreeNodes to CSS.getLayoutTreeAndStyles

third_party/WebKit/Source/platform/inspector_protocol/lib/Array_h.template

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef {{"_".join(config.protocol.namespace)}}_Array_h
 #define {{"_".join(config.protocol.namespace)}}_Array_h
 
 //#include "ErrorSupport.h"
 //#include "Forward.h"
 //#include "ValueConversions.h"
@@ skipping 24 matching lines @@
             errors->setName(StringUtil::fromInteger(i));
             std::unique_ptr<T> item = ValueConversions<T>::parse(array->at(i), errors);
             result->m_vector.push_back(std::move(item));
         }
         errors->pop();
         if (errors->hasErrors())
             return nullptr;
         return result;
     }
 
+    Array() {}

[dgozman, 2016/10/12 21:00:40]

Why not Array::create() defined above?

[alex clarke (OOO till 29th), 2016/10/13 20:25:10]

I'm assuming you don't want this to be public?

[dgozman, 2016/10/14 01:32:33]

Yes.

+    explicit Array(std::vector<std::unique_ptr<T>> vector) : m_vector(std::move(vector)) { }

[dgozman, 2016/10/12 21:00:40]

Let's do
static std::unique_ptr<Array<T>> fromVector(...)
instead.

[alex clarke (OOO till 29th), 2016/10/13 20:25:10]

Done.

+
     void addItem(std::unique_ptr<T> value)
     {
         m_vector.push_back(std::move(value));
     }
 
     size_t length()
     {
         return m_vector.size();
     }
 
@@ skipping 72 matching lines @@
 template<> class Array<String> : public ArrayBase<String> {};
 template<> class Array<int> : public ArrayBase<int> {};
 template<> class Array<double> : public ArrayBase<double> {};
 template<> class Array<bool> : public ArrayBase<bool> {};
 
 {% for namespace in config.protocol.namespace %}
 } // namespace {{namespace}}
 {% endfor %}
 
 #endif // !defined({{"_".join(config.protocol.namespace)}}_Array_h)