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

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: Assure |last_update_check| is set a value Created 4 years, 11 months 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 323 matching lines...) Expand 10 before | Expand all | Expand 10 after
334 error_message); 334 error_message);
335 return; 335 return;
336 } 336 }
337 337
338 if (!CheckPathRestriction(request)) 338 if (!CheckPathRestriction(request))
339 return; 339 return;
340 340
341 version_->SetMainScriptHttpResponseInfo(net_request_->response_info()); 341 version_->SetMainScriptHttpResponseInfo(net_request_->response_info());
342 } 342 }
343 343
344 if (net_request_->response_info().network_accessed) 344 if (net_request_->response_info().network_accessed &&
345 !(net_request_->response_info().was_cached)) {
345 version_->embedded_worker()->OnNetworkAccessedForScriptLoad(); 346 version_->embedded_worker()->OnNetworkAccessedForScriptLoad();
347 }
346 348
347 http_info_.reset(new net::HttpResponseInfo(net_request_->response_info())); 349 http_info_.reset(new net::HttpResponseInfo(net_request_->response_info()));
348 scoped_refptr<HttpResponseInfoIOBuffer> info_buffer = 350 scoped_refptr<HttpResponseInfoIOBuffer> info_buffer =
349 new HttpResponseInfoIOBuffer( 351 new HttpResponseInfoIOBuffer(
350 new net::HttpResponseInfo(net_request_->response_info())); 352 new net::HttpResponseInfo(net_request_->response_info()));
351 net::Error error = cache_writer_->MaybeWriteHeaders( 353 net::Error error = cache_writer_->MaybeWriteHeaders(
352 info_buffer.get(), 354 info_buffer.get(),
353 base::Bind(&ServiceWorkerWriteToCacheJob::OnWriteHeadersComplete, 355 base::Bind(&ServiceWorkerWriteToCacheJob::OnWriteHeadersComplete,
354 weak_factory_.GetWeakPtr())); 356 weak_factory_.GetWeakPtr()));
355 if (error == net::ERR_IO_PENDING) 357 if (error == net::ERR_IO_PENDING)
(...skipping 139 matching lines...) Expand 10 before | Expand all | Expand 10 after
495 } 497 }
496 return context_->storage()->CreateResponseReader(incumbent_resource_id_); 498 return context_->storage()->CreateResponseReader(incumbent_resource_id_);
497 } 499 }
498 500
499 scoped_ptr<ServiceWorkerResponseWriter> 501 scoped_ptr<ServiceWorkerResponseWriter>
500 ServiceWorkerWriteToCacheJob::CreateCacheResponseWriter() { 502 ServiceWorkerWriteToCacheJob::CreateCacheResponseWriter() {
501 return context_->storage()->CreateResponseWriter(resource_id_); 503 return context_->storage()->CreateResponseWriter(resource_id_);
502 } 504 }
503 505
504 } // namespace content 506 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698