[reland] SegmentedString::push() should always push a char in front

Source/core/html/parser/HTMLEntityParserTest.cpp

Index: Source/core/html/parser/HTMLEntityParserTest.cpp
diff --git a/Source/core/html/parser/HTMLEntityParserTest.cpp b/Source/core/html/parser/HTMLEntityParserTest.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..be460d97af72c851561e9cf50b447659489f40bf
--- /dev/null
+++ b/Source/core/html/parser/HTMLEntityParserTest.cpp
@@ -0,0 +1,27 @@
+// Copyright 2015 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.
+
+#include "config.h"
+#include "core/html/parser/HTMLEntityParser.h"
+
+#include <gtest/gtest.h>
+
+namespace blink {
+
+TEST(HTMLEntityParserTest, ConsumeHTMLEntityIncomplete)
+{
+    String original("am"); // Incomplete by purpose.
+    SegmentedString src(original);
+
+    DecodedHTMLEntity entity;
+    bool notEnoughCharacters;
+    bool success = consumeHTMLEntity(src, entity, notEnoughCharacters);
+    EXPECT_TRUE(notEnoughCharacters);
+    EXPECT_FALSE(success);
+
+    // consumeHTMLEntity should recover the original SegmentedString state if failed.
+    EXPECT_STREQ(original.ascii().data(), src.toString().ascii().data());

[yosin_UTC9, 2015/08/27 12:50:14]

nit: EXPECT_EQ() can accept String. So, you can write
EXPECT_EQ(original, src.toString())

+}
+
+} // namespace blink