Remove EmbeddedWorkerInstance::OnScriptRead{Started,Finished}

These are no longer used.

Bug: N/A
Change-Id: I94875f7a7145751b8934c36cce8460ed1f36302f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1919429
Commit-Queue: Kenichi Ishibashi <bashi@chromium.org>
Reviewed-by: Makoto Shimazu <shimazu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#715956}

content/browser/service_worker/embedded_worker_instance.cc

@@ -281,8 +281,6 @@ bool HasSentStartWorker(EmbeddedWorkerInstance::StartingPhase phase) {
       return false;
     case EmbeddedWorkerInstance::SENT_START_WORKER:
     case EmbeddedWorkerInstance::SCRIPT_DOWNLOADING:
-    case EmbeddedWorkerInstance::SCRIPT_READ_STARTED:
-    case EmbeddedWorkerInstance::SCRIPT_READ_FINISHED:
     case EmbeddedWorkerInstance::SCRIPT_STREAMING:
     case EmbeddedWorkerInstance::SCRIPT_LOADED:
     case EmbeddedWorkerInstance::SCRIPT_EVALUATION:
@@ -907,14 +905,6 @@ void EmbeddedWorkerInstance::OnReadyForInspection(
                                                   std::move(host_receiver));
 }
 
-void EmbeddedWorkerInstance::OnScriptReadStarted() {
-  starting_phase_ = SCRIPT_READ_STARTED;
-}
-
-void EmbeddedWorkerInstance::OnScriptReadFinished() {
-  starting_phase_ = SCRIPT_READ_FINISHED;
-}
-
 void EmbeddedWorkerInstance::OnScriptLoaded() {
   if (!inflight_start_task_)
     return;
@@ -1242,10 +1232,6 @@ std::string EmbeddedWorkerInstance::StartingPhaseToString(StartingPhase phase) {
       return "Script downloading";
     case SCRIPT_LOADED:
       return "Script loaded";
-    case SCRIPT_READ_STARTED:
-      return "Script read started";
-    case SCRIPT_READ_FINISHED:
-      return "Script read finished";
     case SCRIPT_STREAMING:
       return "Script streaming";
     case SCRIPT_EVALUATION:

content/browser/service_worker/embedded_worker_instance.h

@@ -73,10 +73,8 @@ class CONTENT_EXPORT EmbeddedWorkerInstance
     SCRIPT_LOADED = 5,
     // SCRIPT_EVALUATED = 6,  // Obsolete
     // THREAD_STARTED = 7,  // Obsolete
-    // Script read happens after SENT_START_WORKER and before SCRIPT_LOADED
-    // (installed scripts only)
-    SCRIPT_READ_STARTED = 8,
-    SCRIPT_READ_FINISHED = 9,
+    // SCRIPT_READ_STARTED = 8,  // Obsolete
+    // SCRIPT_READ_FINISHED = 9,  // Obsolete
     SCRIPT_STREAMING = 10,
     SCRIPT_EVALUATION = 11,
     // Add new values here and update enums.xml.
@@ -195,11 +193,6 @@ class CONTENT_EXPORT EmbeddedWorkerInstance
   // Called when the main script load accessed the network.
   void OnNetworkAccessedForScriptLoad();
 
-  // Called when reading the main script from the service worker script cache
-  // begins and ends.
-  void OnScriptReadStarted();
-  void OnScriptReadFinished();
-
   // Called when the worker is installed.
   void OnWorkerVersionInstalled();
 

tools/metrics/histograms/enums.xml

@@ -16583,8 +16583,8 @@ Called by update_net_error_codes.py.-->
   <int value="5" label="SCRIPT_LOADED"/>
   <int value="6" label="SCRIPT_EVALUATED (obsolete)"/>
   <int value="7" label="THREAD_STARTED"/>
-  <int value="8" label="SCRIPT_READ_STARTED"/>
-  <int value="9" label="SCRIPT_READ_FINISHED"/>
+  <int value="8" label="SCRIPT_READ_STARTED (obsolete)"/>
+  <int value="9" label="SCRIPT_READ_FINISHED (obsolete)"/>
   <int value="10" label="SCRIPT_STREAMING"/>
   <int value="11" label="SCRIPT_EVALUATION"/>
 </enum>