[EventSource] retry field including non-digit chars should be ignored

third_party/WebKit/Source/core/page/EventSourceTest.cpp

 // 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.
 
 #include "core/page/EventSource.h"
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/ScriptState.h"
 #include "bindings/core/v8/ScriptValue.h"
 #include "bindings/core/v8/SerializedScriptValue.h"
@@ skipping 255 matching lines @@
     enqueue("retry:123\n");
     EXPECT_NE(EventSource::defaultReconnectDelay, source()->reconnectDelayForTest());
     EXPECT_EQ(123u, source()->reconnectDelayForTest());
 
     enqueue("retry:a0\n");
     EXPECT_EQ(123u, source()->reconnectDelayForTest());
     enqueue("retry:xi\n");
     EXPECT_EQ(123u, source()->reconnectDelayForTest());
 
     enqueue("retry:2a\n");
-    // TODO(yhirano): Should be ignored.
-    // EXPECT_EQ(123u, source()->reconnectDelayForTest());
-    EXPECT_EQ(2u, source()->reconnectDelayForTest());
+    EXPECT_EQ(123u, source()->reconnectDelayForTest());
 
     enqueue("retry:09a\n");
-    // TODO(yhirano): Should be ignored.
-    // EXPECT_EQ(123u, source()->reconnectDelayForTest());
-    EXPECT_EQ(9u, source()->reconnectDelayForTest());
+    EXPECT_EQ(123u, source()->reconnectDelayForTest());
 
     enqueue("retry:1\b\n");
-    // TODO(yhirano): Should be ignored.
-    // EXPECT_EQ(123u, source()->reconnectDelayForTest());
-    EXPECT_EQ(1u, source()->reconnectDelayForTest());
+    EXPECT_EQ(123u, source()->reconnectDelayForTest());
 
     enqueue("retry:  1234\n");
-    // TODO(yhirano): Should be ignored.
-    // EXPECT_EQ(123u, source()->reconnectDelayForTest());
-    EXPECT_EQ(1234u, source()->reconnectDelayForTest());
+    EXPECT_EQ(123u, source()->reconnectDelayForTest());
 
     enqueue("retry:456 \n");
-    // TODO(yhirano): Should be ignored.
-    // EXPECT_EQ(123u, source()->reconnectDelayForTest());
-    EXPECT_EQ(456u, source()->reconnectDelayForTest());
+    EXPECT_EQ(123u, source()->reconnectDelayForTest());
 
     EXPECT_EQ(0u, listener->events().size());
 }
 
 TEST_F(EventSourceTest, UnrecognizedFieldShouldBeIgnored)
 {
     RefPtrWillBeRawPtr<FakeEventListener> listener = FakeEventListener::create();
 
     source()->addEventListener("message", listener);
     enqueue("data:hello\nhoge:fuga\npiyo\n\n");
@@ skipping 119 matching lines @@
     ASSERT_EQ("message", messageListener->events()[0]->type());
     RefPtrWillBeRawPtr<MessageEvent> messageEvent = static_cast<MessageEvent*>(messageListener->events()[0].get());
     ASSERT_EQ(MessageEvent::DataTypeSerializedScriptValue, messageEvent->dataType());
     EXPECT_EQ("bye", dataAsString(scriptState(), messageEvent.get()));
     EXPECT_EQ("8", messageEvent->lastEventId());
 }
 
 } // namespace
 
 } // namespace blink