Revert of Add a parser for Link headers

Revert of Add a parser for Link headers (patchset #8 id:140001 of https://codereview.chromium.org/889883002/)

Reason for revert:
A memory leak was detected by the asan tool.

Original issue's description:
> Add a parser for Link headers
> 
> This CL adds a parser for Link headers, that would enable to support them for various features that can benefit from such support. Resource Hints (and the non-standard hints: dns-prefetch, prefetch, etc) are likely to be the first users.
> 
> BUG=58456
> 
> Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=189352

TBR=pmeenan@google.com,mkwst@chromium.org,yoav@yoav.ws
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=58456
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=189420

[perezju, 2015-02-03 14:58:57]

Created Revert of Add a parser for Link headers

[commit-bot: I haz the power, 2015-02-03 14:59:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/876633009/1

[perezju, 2015-02-03 15:00:42]

Using the address sanitnizer following the instructions here:
http://www.chromium.org/developers/testing/addresssanitizer

when running webkit_unit_tests we obtain:


[ RUN      ] LinkHeaderTest.Basic
=================================================================
==2925==ERROR: AddressSanitizer: heap-buffer-overflow on address 0x60400006d67b
at pc 0x00000424bf22 bp 0x7fffc411c0b0 sp 0x7fffc411c0a8
READ of size 1 at 0x60400006d67b thread T0
#0 0x424bf21 in parseParameterValue<const unsigned char>
third_party/WebKit/Source/core/loader/LinkHeader.cpp:144:9
#1 0x424bf21 in bool blink::LinkHeader::init<unsigned char const>(unsigned char
const*, unsigned int) third_party/WebKit/Source/core/loader/LinkHeader.cpp:176
#2 0x424b2da in blink::LinkHeader::LinkHeader(WTF::String const&)
third_party/WebKit/Source/core/loader/LinkHeader.cpp:193:21
#3 0xada170 in blink::LinkHeaderTest_Basic_Test::TestBody()
third_party/WebKit/Source/core/loader/LinkHeaderTest.cpp:58:9
#4 0x137c909 in HandleExceptionsInMethodIfSupported<testing::Test, void>
testing/gtest/src/gtest.cc:2420:12
#5 0x137c909 in testing::Test::Run() testing/gtest/src/gtest.cc:2436
#6 0x137e5db in testing::TestInfo::Run() testing/gtest/src/gtest.cc:2612:5
#7 0x137f825 in testing::TestCase::Run() testing/gtest/src/gtest.cc:2730:5
#8 0x13939ee in testing::internal::UnitTestImpl::RunAllTests()
testing/gtest/src/gtest.cc:4602:11
#9 0x1392fe1 in
HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool>
testing/gtest/src/gtest.cc:2420:12
#10 0x1392fe1 in testing::UnitTest::Run() testing/gtest/src/gtest.cc:4220
#11 0x132dd73 in RUN_ALL_TESTS testing/gtest/include/gtest/gtest.h:2326:10
#12 0x132dd73 in base::TestSuite::Run() base/test/test_suite.cc:222
#13 0x124a146 in blink::(anonymous namespace)::runHelper(base::TestSuite*, void
(*)(), void (*)()) third_party/WebKit/Source/web/tests/WebUnitTests.cpp:49:18
#14 0x13258f6 in Run base/callback.h:396:12
#15 0x13258f6 in base::(anonymous
namespace)::LaunchUnitTestsInternal(base::Callback<int ()> const&, int, bool,
base::Callback<void ()> const&) base/test/launcher/unit_test_launcher.cc:181
#16 0x1325468 in base::LaunchUnitTests(int, char**, base::Callback<int ()>
const&) base/test/launcher/unit_test_launcher.cc:423:10
#17 0x124a05d in blink::runWebTests(int, char**, void (*)(), void (*)())
third_party/WebKit/Source/web/tests/WebUnitTests.cpp:66:12
#18 0x7f9c8e6ecec4 in __libc_start_main
/build/buildd/eglibc-2.19/csu/libc-start.c:287

0x60400006d67b is located 0 bytes to the right of 43-byte region
[0x60400006d650,0x60400006d67b)
allocated by thread T0 here:
#0 0x4d91f9 in __interceptor_malloc
(/usr/local/google/code/clankium/src/out_android/Release/webkit_unit_tests+0x4d91f9)
#1 0x169cd51 in partitionAllocGenericFlags
third_party/WebKit/Source/wtf/PartitionAlloc.h:541:20
#2 0x169cd51 in partitionAllocGeneric
third_party/WebKit/Source/wtf/PartitionAlloc.h:557
#3 0x169cd51 in createUninitialized
third_party/WebKit/Source/wtf/text/StringImpl.cpp:299
#4 0x169cd51 in create third_party/WebKit/Source/wtf/text/StringImpl.cpp:418
#5 0x169cd51 in WTF::StringImpl::create(unsigned char const*)
third_party/WebKit/Source/wtf/text/StringImpl.cpp:446
#6 0x16c0c64 in WTF::String::String(char const*)
third_party/WebKit/Source/wtf/text/WTFString.cpp:78:27
#7 0xada165 in blink::LinkHeaderTest_Basic_Test::TestBody()
third_party/WebKit/Source/core/loader/LinkHeaderTest.cpp:58:9
#8 0x137c909 in HandleExceptionsInMethodIfSupported<testing::Test, void>
testing/gtest/src/gtest.cc:2420:12
#9 0x137c909 in testing::Test::Run() testing/gtest/src/gtest.cc:2436
#10 0x137e5db in testing::TestInfo::Run() testing/gtest/src/gtest.cc:2612:5
#11 0x137f825 in testing::TestCase::Run() testing/gtest/src/gtest.cc:2730:5
#12 0x13939ee in testing::internal::UnitTestImpl::RunAllTests()
testing/gtest/src/gtest.cc:4602:11
#13 0x1392fe1 in
HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool>
testing/gtest/src/gtest.cc:2420:12
#14 0x1392fe1 in testing::UnitTest::Run() testing/gtest/src/gtest.cc:4220
#15 0x132dd73 in RUN_ALL_TESTS testing/gtest/include/gtest/gtest.h:2326:10
#16 0x132dd73 in base::TestSuite::Run() base/test/test_suite.cc:222
#17 0x124a146 in blink::(anonymous namespace)::runHelper(base::TestSuite*, void
(*)(), void (*)()) third_party/WebKit/Source/web/tests/WebUnitTests.cpp:49:18
#18 0x13258f6 in Run base/callback.h:396:12
#19 0x13258f6 in base::(anonymous
namespace)::LaunchUnitTestsInternal(base::Callback<int ()> const&, int, bool,
base::Callback<void ()> const&) base/test/launcher/unit_test_launcher.cc:181
#20 0x1325468 in base::LaunchUnitTests(int, char**, base::Callback<int ()>
const&) base/test/launcher/unit_test_launcher.cc:423:10
#21 0x124a05d in blink::runWebTests(int, char**, void (*)(), void (*)())
third_party/WebKit/Source/web/tests/WebUnitTests.cpp:66:12
#22 0x7f9c8e6ecec4 in __libc_start_main
/build/buildd/eglibc-2.19/csu/libc-start.c:287

SUMMARY: AddressSanitizer: heap-buffer-overflow
third_party/WebKit/Source/core/loader/LinkHeader.cpp:144
parseParameterValue<const unsigned char>
Shadow bytes around the buggy address:
0x0c0880005a70: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
0x0c0880005a80: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
0x0c0880005a90: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
0x0c0880005aa0: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
0x0c0880005ab0: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
=>0x0c0880005ac0: fa fa fa fa fa fa fa fa fa fa 00 00 00 00 00[03]
0x0c0880005ad0: fa fa 00 00 00 00 00 00 fa fa fd fd fd fd fd fd
0x0c0880005ae0: fa fa 00 00 00 00 00 00 fa fa fd fd fd fd fd fd
0x0c0880005af0: fa fa 00 00 00 00 00 00 fa fa fd fd fd fd fd fd
0x0c0880005b00: fa fa 00 00 00 00 00 00 fa fa fd fd fd fd fd fd
0x0c0880005b10: fa fa fd fd fd fd fd fa fa fa fd fd fd fd fd fd
Shadow byte legend (one shadow byte represents 8 application bytes):
Addressable:           00
Partially addressable: 01 02 03 04 05 06 07
Heap left redzone:       fa
Heap right redzone:      fb
Freed heap region:       fd
Stack left redzone:      f1
Stack mid redzone:       f2
Stack right redzone:     f3
Stack partial redzone:   f4
Stack after return:      f5
Stack use after scope:   f8
Global redzone:          f9
Global init order:       f6
Poisoned by user:        f7
Container overflow:      fc
Array cookie:            ac
Intra object redzone:    bb
ASan internal:           fe
Left alloca redzone:     ca
Right alloca redzone:    cb
==2925==ABORTING
[780/1761] LinkHeaderTest.Basic (CRASHED)

[commit-bot: I haz the power, 2015-02-03 15:05:32]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=189420

[Mike West, 2015-02-03 15:26:42]

LGTM, thanks for reverting.