Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(163)

Side by Side Diff: content/browser/service_worker/service_worker_write_to_cache_job.cc

Issue 1381153004: Service Worker: Change the criteria for bumping the last update check time (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update the unittest. Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/service_worker/service_worker_write_to_cache_job.h" 5 #include "content/browser/service_worker/service_worker_write_to_cache_job.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/callback.h" 8 #include "base/callback.h"
9 #include "base/strings/stringprintf.h" 9 #include "base/strings/stringprintf.h"
10 #include "base/thread_task_runner_handle.h" 10 #include "base/thread_task_runner_handle.h"
(...skipping 322 matching lines...) Expand 10 before | Expand all | Expand 10 after
333 error_message); 333 error_message);
334 return; 334 return;
335 } 335 }
336 336
337 if (!CheckPathRestriction(request)) 337 if (!CheckPathRestriction(request))
338 return; 338 return;
339 339
340 version_->SetMainScriptHttpResponseInfo(net_request_->response_info()); 340 version_->SetMainScriptHttpResponseInfo(net_request_->response_info());
341 } 341 }
342 342
343 if (net_request_->response_info().network_accessed) 343 if (net_request_->response_info().network_accessed &&
344 !(net_request_->response_info().was_cached)) {
344 version_->embedded_worker()->OnNetworkAccessedForScriptLoad(); 345 version_->embedded_worker()->OnNetworkAccessedForScriptLoad();
346 }
345 347
346 http_info_.reset(new net::HttpResponseInfo(net_request_->response_info())); 348 http_info_.reset(new net::HttpResponseInfo(net_request_->response_info()));
347 scoped_refptr<HttpResponseInfoIOBuffer> info_buffer = 349 scoped_refptr<HttpResponseInfoIOBuffer> info_buffer =
348 new HttpResponseInfoIOBuffer( 350 new HttpResponseInfoIOBuffer(
349 new net::HttpResponseInfo(net_request_->response_info())); 351 new net::HttpResponseInfo(net_request_->response_info()));
350 net::Error error = cache_writer_->MaybeWriteHeaders( 352 net::Error error = cache_writer_->MaybeWriteHeaders(
351 info_buffer.get(), 353 info_buffer.get(),
352 base::Bind(&ServiceWorkerWriteToCacheJob::OnWriteHeadersComplete, 354 base::Bind(&ServiceWorkerWriteToCacheJob::OnWriteHeadersComplete,
353 weak_factory_.GetWeakPtr())); 355 weak_factory_.GetWeakPtr()));
354 if (error == net::ERR_IO_PENDING) 356 if (error == net::ERR_IO_PENDING)
(...skipping 139 matching lines...) Expand 10 before | Expand all | Expand 10 after
494 } 496 }
495 return context_->storage()->CreateResponseReader(incumbent_resource_id_); 497 return context_->storage()->CreateResponseReader(incumbent_resource_id_);
496 } 498 }
497 499
498 scoped_ptr<ServiceWorkerResponseWriter> 500 scoped_ptr<ServiceWorkerResponseWriter>
499 ServiceWorkerWriteToCacheJob::CreateCacheResponseWriter() { 501 ServiceWorkerWriteToCacheJob::CreateCacheResponseWriter() {
500 return context_->storage()->CreateResponseWriter(resource_id_); 502 return context_->storage()->CreateResponseWriter(resource_id_);
501 } 503 }
502 504
503 } // namespace content 505 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698