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

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

Issue 1795863006: service worker: Attribute purpose to start worker attempts for UMA (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: patch for landing? Created 4 years, 9 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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_registration.h" 5 #include "content/browser/service_worker/service_worker_registration.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "content/browser/service_worker/service_worker_context_core.h" 9 #include "content/browser/service_worker/service_worker_context_core.h"
10 #include "content/browser/service_worker/service_worker_info.h" 10 #include "content/browser/service_worker/service_worker_info.h"
(...skipping 269 matching lines...) Expand 10 before | Expand all | Expand 10 after
280 280
281 // "8. Run the [[UpdateState]] algorithm passing registration.activeWorker and 281 // "8. Run the [[UpdateState]] algorithm passing registration.activeWorker and
282 // "activating" as arguments." 282 // "activating" as arguments."
283 activating_version->SetStatus(ServiceWorkerVersion::ACTIVATING); 283 activating_version->SetStatus(ServiceWorkerVersion::ACTIVATING);
284 // "9. Fire a simple event named controllerchange..." 284 // "9. Fire a simple event named controllerchange..."
285 if (activating_version->skip_waiting()) 285 if (activating_version->skip_waiting())
286 FOR_EACH_OBSERVER(Listener, listeners_, OnSkippedWaiting(this)); 286 FOR_EACH_OBSERVER(Listener, listeners_, OnSkippedWaiting(this));
287 287
288 // "10. Queue a task to fire an event named activate..." 288 // "10. Queue a task to fire an event named activate..."
289 activating_version->RunAfterStartWorker( 289 activating_version->RunAfterStartWorker(
290 ServiceWorkerMetrics::EventType::ACTIVATE,
290 base::Bind(&ServiceWorkerRegistration::DispatchActivateEvent, this, 291 base::Bind(&ServiceWorkerRegistration::DispatchActivateEvent, this,
291 activating_version), 292 activating_version),
292 base::Bind(&ServiceWorkerRegistration::OnActivateEventFinished, this, 293 base::Bind(&ServiceWorkerRegistration::OnActivateEventFinished, this,
293 activating_version)); 294 activating_version));
294 } 295 }
295 296
296 void ServiceWorkerRegistration::DeleteVersion( 297 void ServiceWorkerRegistration::DeleteVersion(
297 const scoped_refptr<ServiceWorkerVersion>& version) { 298 const scoped_refptr<ServiceWorkerVersion>& version) {
298 DCHECK_EQ(id(), version->registration_id()); 299 DCHECK_EQ(id(), version->registration_id());
299 300
(...skipping 130 matching lines...) Expand 10 before | Expand all | Expand 10 after
430 if (!context_) { 431 if (!context_) {
431 callback.Run(SERVICE_WORKER_ERROR_ABORT); 432 callback.Run(SERVICE_WORKER_ERROR_ABORT);
432 return; 433 return;
433 } 434 }
434 context_->storage()->NotifyDoneInstallingRegistration( 435 context_->storage()->NotifyDoneInstallingRegistration(
435 this, version.get(), status); 436 this, version.get(), status);
436 callback.Run(status); 437 callback.Run(status);
437 } 438 }
438 439
439 } // namespace content 440 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698