Add begin()/end() to WebVector to enable range-based for loops

Source/web/WebVectorTest.cpp

Index: Source/web/WebVectorTest.cpp
diff --git a/Source/web/WebVectorTest.cpp b/Source/web/WebVectorTest.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..434753753736e9fe9b48337a1f4fe24e08a6d300
--- /dev/null
+++ b/Source/web/WebVectorTest.cpp
@@ -0,0 +1,51 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.

[Peter Beverloo, 2015/03/19 12:03:19]

nit: 2015

[johnme, 2015/03/19 12:47:23]

Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "config.h"
+#include "public/platform/WebVector.h"
+
+#include "wtf/Vector.h"
+#include <gtest/gtest.h>

[Peter Beverloo, 2015/03/19 12:03:19]

nit: no blank line on line 7, add one between 8 and 9.

[johnme, 2015/03/19 12:47:23]

Disagree - other tests I've looked at (WebNodeTest, WebViewTest, WebFrameTest,
DecimalTest, SharedBufferTest) all follow this convention, and it matches the
examples in http://www.chromium.org/blink/coding-style#TOC-include-Statements

+
+namespace blink {
+
+TEST(WebVectorTest, beginend)
+{
+    Vector<int> input;
+    for (int i = 0; i < 5; ++i)
+        input.append(i);
+
+    WebVector<int> webVector(input);
+    ASSERT_EQ(input.size(), webVector.size());
+
+    EXPECT_EQ(webVector.data(), webVector.begin());
+    EXPECT_EQ(webVector.data() + webVector.size(), webVector.end());
+
+    // Use begin()/end() iterators directly.
+    Vector<int> output;
+    for (WebVector<int>::iterator it = webVector.begin(); it != webVector.end(); ++it)
+        output.append(*it);
+    ASSERT_EQ(input.size(), output.size());
+    for (size_t i = 0; i < input.size(); ++i)
+        EXPECT_EQ(input[i], output[i]);
+
+    // Use begin()/end() const_iterators directly.
+    output.clear();
+    const WebVector<int>& constWebVector = webVector;
+    for (WebVector<int>::const_iterator it = constWebVector.begin(); it != constWebVector.end(); ++it)
+        output.append(*it);
+    ASSERT_EQ(input.size(), output.size());
+    for (size_t i = 0; i < input.size(); ++i)
+        EXPECT_EQ(input[i], output[i]);
+
+    // Use range-based for loop.
+    output.clear();
+    for (int x : webVector)
+        output.append(x);
+    ASSERT_EQ(input.size(), output.size());
+    for (size_t i = 0; i < input.size(); ++i)
+        EXPECT_EQ(input[i], output[i]);
+}
+
+} // namespace blink