CHECK that Resource request/response urls are equalIgnoringFragmentIdentifer, not equal

third_party/WebKit/Source/core/fetch/ResourceTest.cpp

Index: third_party/WebKit/Source/core/fetch/ResourceTest.cpp
diff --git a/third_party/WebKit/Source/core/fetch/ResourceTest.cpp b/third_party/WebKit/Source/core/fetch/ResourceTest.cpp
index ce5620e3d70a1ce711c0fe3fb6e5b407fb91aa3e..2abde3ebfdf0ac9162225c74097c198fa35cca6b 100644
--- a/third_party/WebKit/Source/core/fetch/ResourceTest.cpp
+++ b/third_party/WebKit/Source/core/fetch/ResourceTest.cpp
@@ -112,4 +112,22 @@ TEST(ResourceTest, LockFailureNoCrash)
     EXPECT_EQ(size_t(0), resource->encodedSize());
 }
 
+TEST(ResourceTest, RevalidateWithFragment)

[hiroshige, 2016/06/13 20:17:04]

Could you add a comment about what this is testing (i.e. not crashing due to
assertions)?

[yhirano, 2016/06/13 20:17:14]

Can you write some comments that we don't need ASSERT / EXPECT in this test?

[Nate Chapin, 2016/06/13 20:51:21]

Done.

+{
+    KURL url(ParsedURLString, "http://127.0.0.1:8000/foo.html");
+    ResourceResponse response;
+    response.setURL(url);
+    response.setHTTPStatusCode(200);
+    Resource* resource = Resource::create(url, Resource::Raw);
+    resource->responseReceived(response, nullptr);
+    resource->finish();
+
+    url.setFragmentIdentifier("bar");
+    resource->setRevalidatingRequest(ResourceRequest(url));
+    ResourceResponse revalidatingResponse;
+    revalidatingResponse.setURL(url);
+    revalidatingResponse.setHTTPStatusCode(304);
+    resource->responseReceived(revalidatingResponse, nullptr);
+}
+
 } // namespace blink