Fix Link header parsing bug with extension parameter parsing.

Source/core/loader/LinkHeaderTest.cpp

 // 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/loader/LinkHeader.h"
 
 #include <base/macros.h>
 #include <gtest/gtest.h>
 
@@ skipping 39 matching lines @@
         {"< /images/cat.jpg>", "/images/cat.jpg", "", true},
         {"</images/cat.jpg>; rel =", "/images/cat.jpg", "", false},
         {"</images/cat.jpg>; wut=sup; rel =prefetch", "/images/cat.jpg", "prefetch", true},
         {"</images/cat.jpg>; wut=sup ; rel =prefetch", "/images/cat.jpg", "prefetch", true},
         {"</images/cat.jpg>; wut=sup ; rel =prefetch  \t  ;", "/images/cat.jpg", "prefetch", true},
         {"</images/cat.jpg> wut=sup ; rel =prefetch  \t  ;", "/images/cat.jpg", "", false},
         {"<   /images/cat.jpg", "", "", false},
         {"<   http://wut.com/  sdfsdf ?sd>; rel=dns-prefetch", "http://wut.com/", "dns-prefetch", true},
         {"<   http://wut.com/%20%20%3dsdfsdf?sd>; rel=dns-prefetch", "http://wut.com/%20%20%3dsdfsdf?sd", "dns-prefetch", true},
         {"<   http://wut.com/dfsdf?sdf=ghj&wer=rty>; rel=prefetch", "http://wut.com/dfsdf?sdf=ghj&wer=rty", "prefetch", true},
-        {"<   http://wut.com/dfsdf?sdf=ghj&wer=rty>;;;;; rel=prefetch", "http://wut.com/dfsdf?sdf=ghj&wer=rty", "", false},
+        {"<   http://wut.com/dfsdf?sdf=ghj&wer=rty>;;;;; rel=prefetch", "http://wut.com/dfsdf?sdf=ghj&wer=rty", "prefetch", true},
         {"</images/cat.jpg>; anchor=foo; rel=prefetch;", "/images/cat.jpg", "", false},
         {"</images/cat.jpg>; rel=prefetch;anchor=foo ", "/images/cat.jpg", "prefetch", false},
         {"</images/cat.jpg>; anchor='foo'; rel=prefetch;", "/images/cat.jpg", "", false},
         {"</images/cat.jpg>; rel=prefetch;anchor='foo' ", "/images/cat.jpg", "prefetch", false},
         {"</images/cat.jpg>; rel=prefetch;anchor='' ", "/images/cat.jpg", "prefetch", false},
         {"</images/cat.jpg>; rel=prefetch;", "/images/cat.jpg", "prefetch", true},
         {"</images/cat.jpg>; rel=prefetch    ;", "/images/cat.jpg", "prefetch", true},
         {"</images/ca,t.jpg>; rel=prefetch    ;", "/images/ca,t.jpg", "prefetch", true},
         {"<simple.css>; rel=stylesheet; title=\"title with a DQUOTE and backslash\"", "simple.css", "stylesheet", true},
         {"<simple.css>; rel=stylesheet; title=\"title with a DQUOTE \\\" and backslash: \\\"", "simple.css", "stylesheet", false},
         {"<simple.css>; title=\"title with a DQUOTE \\\" and backslash: \"; rel=stylesheet; ", "simple.css", "stylesheet", true},
         {"<simple.css>; title=\'title with a DQUOTE \\\' and backslash: \'; rel=stylesheet; ", "simple.css", "stylesheet", true},
         {"<simple.css>; title=\"title with a DQUOTE \\\" and ;backslash,: \"; rel=stylesheet; ", "simple.css", "stylesheet", true},
         {"<simple.css>; title=\"title with a DQUOTE \' and ;backslash,: \"; rel=stylesheet; ", "simple.css", "stylesheet", true},
         {"<simple.css>; title=\"\"; rel=stylesheet; ", "simple.css", "stylesheet", true},
         {"<simple.css>; title=\"\"; rel=\"stylesheet\"; ", "simple.css", "stylesheet", true},
         {"<simple.css>; rel=stylesheet; title=\"", "simple.css", "stylesheet", false},
         {"<simple.css>; rel=stylesheet; title=\"\"", "simple.css", "stylesheet", true},
         {"<simple.css>; rel=\"stylesheet\"; title=\"", "simple.css", "stylesheet", false},
         {"<simple.css>; rel=\";style,sheet\"; title=\"", "simple.css", ";style,sheet", false},
         {"<simple.css>; rel=\"bla'sdf\"; title=\"", "simple.css", "bla'sdf", false},
         {"<simple.css>; rel=\"\"; title=\"\"", "simple.css", "", true},
         {"<simple.css>; rel=''; title=\"\"", "simple.css", "", true},
+        {"<simple.css>; rel=''; title=", "simple.css", "", false},
+        {"<simple.css>; rel=''; title", "simple.css", "", false},
+        {"<simple.css>; rel=''; media", "simple.css", "", false},
+        {"<simple.css>; rel=''; hreflang", "simple.css", "", false},
+        {"<simple.css>; rel=''; type", "simple.css", "", false},
+        {"<simple.css>; rel=''; rev", "simple.css", "", false},
+        {"<simple.css>; rel=''; bla", "simple.css", "", true},
         {"<simple.css>; rel='prefetch", "simple.css", "", false},
         {"<simple.css>; rel=\"prefetch", "simple.css", "", false},
         {"<simple.css>; rel=\"", "simple.css", "", false},
     };
 
 
     // Test the cases with a single header
     for (auto& testCase : cases) {
         LinkHeaderSet headerSet(testCase.headerValue);
         LinkHeader& header = headerSet[0];
         ASSERT_STREQ(testCase.url, header.url().ascii().data());
         ASSERT_STREQ(testCase.rel, header.rel().ascii().data());
         ASSERT_EQ(testCase.valid, header.valid());
     }
 }
 
 TEST(LinkHeaderTest, Double)
 {
     struct DoubleTestCase {
         const char* headerValue;
         const char* url;
         const char* rel;
         bool valid;
         const char* url2;
         const char* rel2;
         bool valid2;
     } cases[] = {
         {"<ybg.css>; rel=stylesheet, <simple.css>; rel=stylesheet", "ybg.css", "stylesheet", true, "simple.css", "stylesheet", true},
+        {"<ybg.css>; rel=stylesheet,<simple.css>; rel=stylesheet", "ybg.css", "stylesheet", true, "simple.css", "stylesheet", true},
+        {"<ybg.css>; rel=stylesheet;crossorigin,<simple.css>; rel=stylesheet", "ybg.css", "stylesheet", true, "simple.css", "stylesheet", true},
     };
 
     for (auto& testCase : cases) {
         LinkHeaderSet headerSet(testCase.headerValue);
         LinkHeader& header1 = headerSet[0];
         LinkHeader& header2 = headerSet[1];
         ASSERT_STREQ(testCase.url, header1.url().ascii().data());
         ASSERT_STREQ(testCase.rel, header1.rel().ascii().data());
         ASSERT_EQ(testCase.valid, header1.valid());
         ASSERT_STREQ(testCase.url2, header2.url().ascii().data());
         ASSERT_STREQ(testCase.rel2, header2.rel().ascii().data());
         ASSERT_EQ(testCase.valid2, header2.valid());
     }
 }
 
 TEST(LinkHeaderTest, CrossOrigin)
 {
     struct TestCase {
         const char* headerValue;
         const char* url;
         const char* rel;
         const CrossOriginAttributeValue crossorigin;
         bool valid;
     } cases[] = {
         {"<http://whatever.com>; rel=preconnect", "http://whatever.com", "preconnect", CrossOriginAttributeNotSet, true},
         {"<http://whatever.com>; rel=preconnect; crossorigin=", "http://whatever.com", "preconnect", CrossOriginAttributeAnonymous, true},
         {"<http://whatever.com>; rel=preconnect; crossorigin", "http://whatever.com", "preconnect", CrossOriginAttributeAnonymous, true},
+        {"<http://whatever.com>; rel=preconnect; crossorigin;", "http://whatever.com", "preconnect", CrossOriginAttributeAnonymous, true},
+        {"<http://whatever.com>; rel=preconnect; crossorigin, <http://whatever2.com>; rel=preconnect", "http://whatever.com", "preconnect", CrossOriginAttributeAnonymous, true},
+        {"<http://whatever.com>; rel=preconnect; crossorigin,<http://whatever2.com>; rel=preconnect", "http://whatever.com", "preconnect", CrossOriginAttributeAnonymous, true},
         {"<http://whatever.com>; rel=preconnect; crossorigin=anonymous", "http://whatever.com", "preconnect", CrossOriginAttributeAnonymous, true},
         {"<http://whatever.com>; rel=preconnect; crossorigin=use-credentials", "http://whatever.com", "preconnect", CrossOriginAttributeUseCredentials, true},
         {"<http://whatever.com>; rel=preconnect; crossorigin=whatever", "http://whatever.com", "preconnect", CrossOriginAttributeAnonymous,  true},
+        {"<http://whatever.com>; rel=preconnect; crossorig|in=whatever", "http://whatever.com", "preconnect", CrossOriginAttributeNotSet,  true},
+        {"<http://whatever.com>; rel=preconnect; crossorigin|=whatever", "http://whatever.com", "preconnect", CrossOriginAttributeNotSet,  true},
+        {"<http://whatever.com>; rel=preconnect; valid!", "http://whatever.com", "preconnect", CrossOriginAttributeNotSet,  true},
+        {"<http://whatever.com>; rel=preconnect; valid$", "http://whatever.com", "preconnect", CrossOriginAttributeNotSet,  true},
+        {"<http://whatever.com>; rel=preconnect; invalid@", "http://whatever.com", "preconnect", CrossOriginAttributeNotSet,  false},
+        {"<http://whatever.com>; rel=preconnect; invalid*", "http://whatever.com", "preconnect", CrossOriginAttributeNotSet,  false},
     };
 
 
     // Test the cases with a single header
     for (auto& testCase : cases) {
         LinkHeaderSet headerSet(testCase.headerValue);
         LinkHeader& header = headerSet[0];
         ASSERT_STREQ(testCase.url, header.url().ascii().data());
         ASSERT_STREQ(testCase.rel, header.rel().ascii().data());
         ASSERT_EQ(testCase.crossorigin, header.crossOrigin());
         ASSERT_EQ(testCase.valid, header.valid());
     }
 }
 } // namespace blink