Update installed ServiceWorkers.

content/browser/service_worker/service_worker_register_job_base.h

Index: content/browser/service_worker/service_worker_register_job_base.h
diff --git a/content/browser/service_worker/service_worker_register_job_base.h b/content/browser/service_worker/service_worker_register_job_base.h
index 7995adda7c68294f9cf53bef9ed5b01264149831..9233e7c69af1a079053142c4d7f1f435b25f0533 100644
--- a/content/browser/service_worker/service_worker_register_job_base.h
+++ b/content/browser/service_worker/service_worker_register_job_base.h
@@ -7,11 +7,13 @@
 
 namespace content {
 

[falken, 2014/07/17 06:33:41]

I think some comment is still useful. "Base class for register/update/unregister
jobs managed by ServiceWorkerJobCoordinator."? 

[michaeln, 2014/07/18 03:40:48]

pet peevish, but i think we have too many comments stating the obvious (like the
comment on the GetType() method which  says returns the type of this job)

when there are too many comments that aren't needed, the comments that are
needed go unseen and unread because the readers expect them to not be useful

[falken, 2014/07/18 05:13:24]

Yep I definitely agree about unneeded comments. I find class comments useful
though, especially for stating (even briefly) the relationship between classes:
this "job" is the same "job" in the name JobCoordinator. When I started on SW I
wished more class comments like that existed.

The GetType() comment I agree is a bit silly and normally would have omitted it.
But I try to add a comment for functions that are part of a public interface
that other classes must implement.

Anyway, it's up to you here.

-// A base class for ServiceWorkerRegisterJob and ServiceWorkerUnregisterJob. A
-// job lives only for the lifetime of a single registration or unregistration.
 class ServiceWorkerRegisterJobBase {
  public:
-  enum RegistrationJobType { REGISTRATION, UNREGISTRATION, };
+  enum RegistrationJobType {
+    REGISTRATION_JOB,
+    UNREGISTRATION_JOB,
+    UPDATE_JOB
+  };
 
   virtual ~ServiceWorkerRegisterJobBase() {}