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

third_party/WebKit/Source/core/fetch/ResourceTest.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 "core/fetch/Resource.h"
 
 #include "core/fetch/MemoryCache.h"
 #include "platform/SharedBuffer.h"
 #include "platform/network/ResourceRequest.h"
 #include "platform/network/ResourceResponse.h"
@@ skipping 94 matching lines @@
 
     resource->finish(currentTime());
     resource->prune();
     ASSERT_TRUE(resource->isPurgeable());
     bool didLock = resource->lock();
     ASSERT_FALSE(didLock);
     EXPECT_EQ(nullptr, resource->resourceBuffer());
     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