| Index: third_party/WebKit/Source/core/fetch/CachingCorrectnessTest.cpp
|
| diff --git a/third_party/WebKit/Source/core/fetch/CachingCorrectnessTest.cpp b/third_party/WebKit/Source/core/fetch/CachingCorrectnessTest.cpp
|
| index 5cdd7399a4d3b34552706fe171b2ebc7a491914c..dfabd5005113ba7efca422afce13d2676a5bb7b0 100644
|
| --- a/third_party/WebKit/Source/core/fetch/CachingCorrectnessTest.cpp
|
| +++ b/third_party/WebKit/Source/core/fetch/CachingCorrectnessTest.cpp
|
| @@ -76,11 +76,11 @@ protected:
|
| m_proxyPlatform.advanceClock(seconds);
|
| }
|
|
|
| - PassRefPtrWillBeRawPtr<Resource> resourceFromResourceResponse(ResourceResponse response, Resource::Type type = Resource::Raw)
|
| + RawPtr<Resource> resourceFromResourceResponse(ResourceResponse response, Resource::Type type = Resource::Raw)
|
| {
|
| if (response.url().isNull())
|
| response.setURL(KURL(ParsedURLString, kResourceURL));
|
| - RefPtrWillBeRawPtr<Resource> resource = nullptr;
|
| + RawPtr<Resource> resource = nullptr;
|
| switch (type) {
|
| case Resource::Raw:
|
| resource = Resource::create(ResourceRequest(response.url()), type);
|
| @@ -98,11 +98,11 @@ protected:
|
| return resource;
|
| }
|
|
|
| - PassRefPtrWillBeRawPtr<Resource> resourceFromResourceRequest(ResourceRequest request, Resource::Type type = Resource::Raw)
|
| + RawPtr<Resource> resourceFromResourceRequest(ResourceRequest request, Resource::Type type = Resource::Raw)
|
| {
|
| if (request.url().isNull())
|
| request.setURL(KURL(ParsedURLString, kResourceURL));
|
| - RefPtrWillBeRawPtr<Resource> resource =
|
| + RawPtr<Resource> resource =
|
| Resource::create(request, type);
|
| resource->setResponse(ResourceResponse(KURL(ParsedURLString, kResourceURL), "text/html", 0, nullAtom, String()));
|
| memoryCache()->add(resource.get());
|
| @@ -110,13 +110,13 @@ protected:
|
| return resource;
|
| }
|
|
|
| - PassRefPtrWillBeRawPtr<Resource> fetch()
|
| + RawPtr<Resource> fetch()
|
| {
|
| FetchRequest fetchRequest(ResourceRequest(KURL(ParsedURLString, kResourceURL)), FetchInitiatorInfo());
|
| return RawResource::fetchSynchronously(fetchRequest, fetcher());
|
| }
|
|
|
| - PassRefPtrWillBeRawPtr<Resource> fetchImage()
|
| + RawPtr<Resource> fetchImage()
|
| {
|
| FetchRequest fetchRequest(ResourceRequest(KURL(ParsedURLString, kResourceURL)), FetchInitiatorInfo());
|
| return ImageResource::fetch(fetchRequest, fetcher());
|
| @@ -174,12 +174,12 @@ TEST_F(CachingCorrectnessTest, FreshFromLastModified)
|
| fresh200Response.setHTTPHeaderField("Date", kOriginalRequestDateAsString);
|
| fresh200Response.setHTTPHeaderField("Last-Modified", kOneDayBeforeOriginalRequest);
|
|
|
| - RefPtrWillBeRawPtr<Resource> fresh200 = resourceFromResourceResponse(fresh200Response);
|
| + RawPtr<Resource> fresh200 = resourceFromResourceResponse(fresh200Response);
|
|
|
| // Advance the clock within the implicit freshness period of this resource before we make a request.
|
| advanceClock(600.);
|
|
|
| - RefPtrWillBeRawPtr<Resource> fetched = fetch();
|
| + RawPtr<Resource> fetched = fetch();
|
| EXPECT_EQ(fresh200, fetched);
|
| }
|
|
|
| @@ -190,12 +190,12 @@ TEST_F(CachingCorrectnessTest, FreshFromExpires)
|
| fresh200Response.setHTTPHeaderField("Date", kOriginalRequestDateAsString);
|
| fresh200Response.setHTTPHeaderField("Expires", kOneDayAfterOriginalRequest);
|
|
|
| - RefPtrWillBeRawPtr<Resource> fresh200 = resourceFromResourceResponse(fresh200Response);
|
| + RawPtr<Resource> fresh200 = resourceFromResourceResponse(fresh200Response);
|
|
|
| // Advance the clock within the freshness period of this resource before we make a request.
|
| advanceClock(24. * 60. * 60. - 15.);
|
|
|
| - RefPtrWillBeRawPtr<Resource> fetched = fetch();
|
| + RawPtr<Resource> fetched = fetch();
|
| EXPECT_EQ(fresh200, fetched);
|
| }
|
|
|
| @@ -206,12 +206,12 @@ TEST_F(CachingCorrectnessTest, FreshFromMaxAge)
|
| fresh200Response.setHTTPHeaderField("Date", kOriginalRequestDateAsString);
|
| fresh200Response.setHTTPHeaderField("Cache-Control", "max-age=600");
|
|
|
| - RefPtrWillBeRawPtr<Resource> fresh200 = resourceFromResourceResponse(fresh200Response);
|
| + RawPtr<Resource> fresh200 = resourceFromResourceResponse(fresh200Response);
|
|
|
| // Advance the clock within the freshness period of this resource before we make a request.
|
| advanceClock(500.);
|
|
|
| - RefPtrWillBeRawPtr<Resource> fetched = fetch();
|
| + RawPtr<Resource> fetched = fetch();
|
| EXPECT_EQ(fresh200, fetched);
|
| }
|
|
|
| @@ -223,12 +223,12 @@ TEST_F(CachingCorrectnessTest, DISABLED_ExpiredFromLastModified)
|
| expired200Response.setHTTPHeaderField("Date", kOriginalRequestDateAsString);
|
| expired200Response.setHTTPHeaderField("Last-Modified", kOneDayBeforeOriginalRequest);
|
|
|
| - RefPtrWillBeRawPtr<Resource> expired200 = resourceFromResourceResponse(expired200Response);
|
| + RawPtr<Resource> expired200 = resourceFromResourceResponse(expired200Response);
|
|
|
| // Advance the clock beyond the implicit freshness period.
|
| advanceClock(24. * 60. * 60. * 0.2);
|
|
|
| - RefPtrWillBeRawPtr<Resource> fetched = fetch();
|
| + RawPtr<Resource> fetched = fetch();
|
| EXPECT_NE(expired200, fetched);
|
| }
|
|
|
| @@ -239,12 +239,12 @@ TEST_F(CachingCorrectnessTest, ExpiredFromExpires)
|
| expired200Response.setHTTPHeaderField("Date", kOriginalRequestDateAsString);
|
| expired200Response.setHTTPHeaderField("Expires", kOneDayAfterOriginalRequest);
|
|
|
| - RefPtrWillBeRawPtr<Resource> expired200 = resourceFromResourceResponse(expired200Response);
|
| + RawPtr<Resource> expired200 = resourceFromResourceResponse(expired200Response);
|
|
|
| // Advance the clock within the expiredness period of this resource before we make a request.
|
| advanceClock(24. * 60. * 60. + 15.);
|
|
|
| - RefPtrWillBeRawPtr<Resource> fetched = fetch();
|
| + RawPtr<Resource> fetched = fetch();
|
| EXPECT_NE(expired200, fetched);
|
| }
|
|
|
| @@ -256,12 +256,12 @@ TEST_F(CachingCorrectnessTest, NewImageExpiredFromExpires)
|
| expired200Response.setHTTPHeaderField("Date", kOriginalRequestDateAsString);
|
| expired200Response.setHTTPHeaderField("Expires", kOneDayAfterOriginalRequest);
|
|
|
| - RefPtrWillBeRawPtr<Resource> expired200 = resourceFromResourceResponse(expired200Response, Resource::Image);
|
| + RawPtr<Resource> expired200 = resourceFromResourceResponse(expired200Response, Resource::Image);
|
|
|
| // Advance the clock within the expiredness period of this resource before we make a request.
|
| advanceClock(24. * 60. * 60. + 15.);
|
|
|
| - RefPtrWillBeRawPtr<Resource> fetched = fetchImage();
|
| + RawPtr<Resource> fetched = fetchImage();
|
| EXPECT_NE(expired200, fetched);
|
| }
|
|
|
| @@ -274,17 +274,17 @@ TEST_F(CachingCorrectnessTest, ReuseImageExpiredFromExpires)
|
| expired200Response.setHTTPHeaderField("Date", kOriginalRequestDateAsString);
|
| expired200Response.setHTTPHeaderField("Expires", kOneDayAfterOriginalRequest);
|
|
|
| - RefPtrWillBeRawPtr<Resource> expired200 = resourceFromResourceResponse(expired200Response, Resource::Image);
|
| + RawPtr<Resource> expired200 = resourceFromResourceResponse(expired200Response, Resource::Image);
|
|
|
| // Advance the clock within the freshness period, and make a request to add this image to the document resources.
|
| advanceClock(15.);
|
| - RefPtrWillBeRawPtr<Resource> firstFetched = fetchImage();
|
| + RawPtr<Resource> firstFetched = fetchImage();
|
| EXPECT_EQ(expired200, firstFetched);
|
|
|
| // Advance the clock within the expiredness period of this resource before we make a request.
|
| advanceClock(24. * 60. * 60. + 15.);
|
|
|
| - RefPtrWillBeRawPtr<Resource> fetched = fetchImage();
|
| + RawPtr<Resource> fetched = fetchImage();
|
| EXPECT_EQ(expired200, fetched);
|
| }
|
|
|
| @@ -295,12 +295,12 @@ TEST_F(CachingCorrectnessTest, ExpiredFromMaxAge)
|
| expired200Response.setHTTPHeaderField("Date", kOriginalRequestDateAsString);
|
| expired200Response.setHTTPHeaderField("Cache-Control", "max-age=600");
|
|
|
| - RefPtrWillBeRawPtr<Resource> expired200 = resourceFromResourceResponse(expired200Response);
|
| + RawPtr<Resource> expired200 = resourceFromResourceResponse(expired200Response);
|
|
|
| // Advance the clock within the expiredness period of this resource before we make a request.
|
| advanceClock(700.);
|
|
|
| - RefPtrWillBeRawPtr<Resource> fetched = fetch();
|
| + RawPtr<Resource> fetched = fetch();
|
| EXPECT_NE(expired200, fetched);
|
| }
|
|
|
| @@ -312,12 +312,12 @@ TEST_F(CachingCorrectnessTest, FreshButNoCache)
|
| fresh200NocacheResponse.setHTTPHeaderField(HTTPNames::Expires, kOneDayAfterOriginalRequest);
|
| fresh200NocacheResponse.setHTTPHeaderField(HTTPNames::Cache_Control, "no-cache");
|
|
|
| - RefPtrWillBeRawPtr<Resource> fresh200Nocache = resourceFromResourceResponse(fresh200NocacheResponse);
|
| + RawPtr<Resource> fresh200Nocache = resourceFromResourceResponse(fresh200NocacheResponse);
|
|
|
| // Advance the clock within the freshness period of this resource before we make a request.
|
| advanceClock(24. * 60. * 60. - 15.);
|
|
|
| - RefPtrWillBeRawPtr<Resource> fetched = fetch();
|
| + RawPtr<Resource> fetched = fetch();
|
| EXPECT_NE(fresh200Nocache, fetched);
|
| }
|
|
|
| @@ -325,8 +325,8 @@ TEST_F(CachingCorrectnessTest, RequestWithNoCahe)
|
| {
|
| ResourceRequest noCacheRequest;
|
| noCacheRequest.setHTTPHeaderField(HTTPNames::Cache_Control, "no-cache");
|
| - RefPtrWillBeRawPtr<Resource> noCacheResource = resourceFromResourceRequest(noCacheRequest);
|
| - RefPtrWillBeRawPtr<Resource> fetched = fetch();
|
| + RawPtr<Resource> noCacheResource = resourceFromResourceRequest(noCacheRequest);
|
| + RawPtr<Resource> fetched = fetch();
|
| EXPECT_NE(noCacheResource, fetched);
|
| }
|
|
|
| @@ -338,12 +338,12 @@ TEST_F(CachingCorrectnessTest, FreshButNoStore)
|
| fresh200NostoreResponse.setHTTPHeaderField(HTTPNames::Expires, kOneDayAfterOriginalRequest);
|
| fresh200NostoreResponse.setHTTPHeaderField(HTTPNames::Cache_Control, "no-store");
|
|
|
| - RefPtrWillBeRawPtr<Resource> fresh200Nostore = resourceFromResourceResponse(fresh200NostoreResponse);
|
| + RawPtr<Resource> fresh200Nostore = resourceFromResourceResponse(fresh200NostoreResponse);
|
|
|
| // Advance the clock within the freshness period of this resource before we make a request.
|
| advanceClock(24. * 60. * 60. - 15.);
|
|
|
| - RefPtrWillBeRawPtr<Resource> fetched = fetch();
|
| + RawPtr<Resource> fetched = fetch();
|
| EXPECT_NE(fresh200Nostore, fetched);
|
| }
|
|
|
| @@ -351,8 +351,8 @@ TEST_F(CachingCorrectnessTest, RequestWithNoStore)
|
| {
|
| ResourceRequest noStoreRequest;
|
| noStoreRequest.setHTTPHeaderField(HTTPNames::Cache_Control, "no-store");
|
| - RefPtrWillBeRawPtr<Resource> noStoreResource = resourceFromResourceRequest(noStoreRequest);
|
| - RefPtrWillBeRawPtr<Resource> fetched = fetch();
|
| + RawPtr<Resource> noStoreResource = resourceFromResourceRequest(noStoreRequest);
|
| + RawPtr<Resource> fetched = fetch();
|
| EXPECT_NE(noStoreResource, fetched);
|
| }
|
|
|
| @@ -366,12 +366,12 @@ TEST_F(CachingCorrectnessTest, DISABLED_FreshButMustRevalidate)
|
| fresh200MustRevalidateResponse.setHTTPHeaderField(HTTPNames::Expires, kOneDayAfterOriginalRequest);
|
| fresh200MustRevalidateResponse.setHTTPHeaderField(HTTPNames::Cache_Control, "must-revalidate");
|
|
|
| - RefPtrWillBeRawPtr<Resource> fresh200MustRevalidate = resourceFromResourceResponse(fresh200MustRevalidateResponse);
|
| + RawPtr<Resource> fresh200MustRevalidate = resourceFromResourceResponse(fresh200MustRevalidateResponse);
|
|
|
| // Advance the clock within the freshness period of this resource before we make a request.
|
| advanceClock(24. * 60. * 60. - 15.);
|
|
|
| - RefPtrWillBeRawPtr<Resource> fetched = fetch();
|
| + RawPtr<Resource> fetched = fetch();
|
| EXPECT_NE(fresh200MustRevalidate, fetched);
|
| }
|
|
|
| @@ -381,7 +381,7 @@ TEST_F(CachingCorrectnessTest, FreshWithFreshRedirect)
|
| const char redirectTargetUrlString[] = "http://redirect-target.com";
|
| KURL redirectTargetUrl(ParsedURLString, redirectTargetUrlString);
|
|
|
| - RefPtrWillBeRawPtr<Resource> firstResource = Resource::create(ResourceRequest(redirectUrl), Resource::Raw);
|
| + RawPtr<Resource> firstResource = Resource::create(ResourceRequest(redirectUrl), Resource::Raw);
|
|
|
| ResourceResponse fresh301Response;
|
| fresh301Response.setURL(redirectUrl);
|
| @@ -406,7 +406,7 @@ TEST_F(CachingCorrectnessTest, FreshWithFreshRedirect)
|
|
|
| advanceClock(500.);
|
|
|
| - RefPtrWillBeRawPtr<Resource> fetched = fetch();
|
| + RawPtr<Resource> fetched = fetch();
|
| EXPECT_EQ(firstResource, fetched);
|
| }
|
|
|
| @@ -416,7 +416,7 @@ TEST_F(CachingCorrectnessTest, FreshWithStaleRedirect)
|
| const char redirectTargetUrlString[] = "http://redirect-target.com";
|
| KURL redirectTargetUrl(ParsedURLString, redirectTargetUrlString);
|
|
|
| - RefPtrWillBeRawPtr<Resource> firstResource = Resource::create(ResourceRequest(redirectUrl), Resource::Raw);
|
| + RawPtr<Resource> firstResource = Resource::create(ResourceRequest(redirectUrl), Resource::Raw);
|
|
|
| ResourceResponse stale301Response;
|
| stale301Response.setURL(redirectUrl);
|
| @@ -440,7 +440,7 @@ TEST_F(CachingCorrectnessTest, FreshWithStaleRedirect)
|
|
|
| advanceClock(500.);
|
|
|
| - RefPtrWillBeRawPtr<Resource> fetched = fetch();
|
| + RawPtr<Resource> fetched = fetch();
|
| EXPECT_NE(firstResource, fetched);
|
| }
|
|
|
| @@ -448,14 +448,14 @@ TEST_F(CachingCorrectnessTest, PostToSameURLTwice)
|
| {
|
| ResourceRequest request1(KURL(ParsedURLString, kResourceURL));
|
| request1.setHTTPMethod(HTTPNames::POST);
|
| - RefPtrWillBeRawPtr<Resource> resource1 = Resource::create(ResourceRequest(request1.url()), Resource::Raw);
|
| + RawPtr<Resource> resource1 = Resource::create(ResourceRequest(request1.url()), Resource::Raw);
|
| resource1->setLoading(true);
|
| memoryCache()->add(resource1.get());
|
|
|
| ResourceRequest request2(KURL(ParsedURLString, kResourceURL));
|
| request2.setHTTPMethod(HTTPNames::POST);
|
| FetchRequest fetch2(request2, FetchInitiatorInfo());
|
| - RefPtrWillBeRawPtr<Resource> resource2 = RawResource::fetchSynchronously(fetch2, fetcher());
|
| + RawPtr<Resource> resource2 = RawResource::fetchSynchronously(fetch2, fetcher());
|
|
|
| EXPECT_EQ(resource2, memoryCache()->resourceForURL(request2.url()));
|
| EXPECT_NE(resource1, resource2);
|
| @@ -467,7 +467,7 @@ TEST_F(CachingCorrectnessTest, 302RedirectNotImplicitlyFresh)
|
| const char redirectTargetUrlString[] = "http://redirect-target.com";
|
| KURL redirectTargetUrl(ParsedURLString, redirectTargetUrlString);
|
|
|
| - RefPtrWillBeRawPtr<Resource> firstResource = Resource::create(ResourceRequest(redirectUrl), Resource::Raw);
|
| + RawPtr<Resource> firstResource = Resource::create(ResourceRequest(redirectUrl), Resource::Raw);
|
|
|
| ResourceResponse fresh302Response;
|
| fresh302Response.setURL(redirectUrl);
|
| @@ -492,7 +492,7 @@ TEST_F(CachingCorrectnessTest, 302RedirectNotImplicitlyFresh)
|
|
|
| advanceClock(500.);
|
|
|
| - RefPtrWillBeRawPtr<Resource> fetched = fetch();
|
| + RawPtr<Resource> fetched = fetch();
|
| EXPECT_NE(firstResource, fetched);
|
| }
|
|
|
| @@ -502,7 +502,7 @@ TEST_F(CachingCorrectnessTest, 302RedirectExplicitlyFreshMaxAge)
|
| const char redirectTargetUrlString[] = "http://redirect-target.com";
|
| KURL redirectTargetUrl(ParsedURLString, redirectTargetUrlString);
|
|
|
| - RefPtrWillBeRawPtr<Resource> firstResource = Resource::create(ResourceRequest(redirectUrl), Resource::Raw);
|
| + RawPtr<Resource> firstResource = Resource::create(ResourceRequest(redirectUrl), Resource::Raw);
|
|
|
| ResourceResponse fresh302Response;
|
| fresh302Response.setURL(redirectUrl);
|
| @@ -527,7 +527,7 @@ TEST_F(CachingCorrectnessTest, 302RedirectExplicitlyFreshMaxAge)
|
|
|
| advanceClock(500.);
|
|
|
| - RefPtrWillBeRawPtr<Resource> fetched = fetch();
|
| + RawPtr<Resource> fetched = fetch();
|
| EXPECT_EQ(firstResource, fetched);
|
| }
|
|
|
| @@ -537,7 +537,7 @@ TEST_F(CachingCorrectnessTest, 302RedirectExplicitlyFreshExpires)
|
| const char redirectTargetUrlString[] = "http://redirect-target.com";
|
| KURL redirectTargetUrl(ParsedURLString, redirectTargetUrlString);
|
|
|
| - RefPtrWillBeRawPtr<Resource> firstResource = Resource::create(ResourceRequest(redirectUrl), Resource::Raw);
|
| + RawPtr<Resource> firstResource = Resource::create(ResourceRequest(redirectUrl), Resource::Raw);
|
|
|
| ResourceResponse fresh302Response;
|
| fresh302Response.setURL(redirectUrl);
|
| @@ -562,7 +562,7 @@ TEST_F(CachingCorrectnessTest, 302RedirectExplicitlyFreshExpires)
|
|
|
| advanceClock(500.);
|
|
|
| - RefPtrWillBeRawPtr<Resource> fetched = fetch();
|
| + RawPtr<Resource> fetched = fetch();
|
| EXPECT_EQ(firstResource, fetched);
|
| }
|
|
|
|
|