ServiceWorkerJob::m_lastResponse is unneeded
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 21 Feb 2018 21:51:31 +0000 (21:51 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 21 Feb 2018 21:51:31 +0000 (21:51 +0000)
https://bugs.webkit.org/show_bug.cgi?id=183013

Patch by Youenn Fablet <youenn@apple.com> on 2018-02-21
Reviewed by Chris Dumez.

Remove this field since not used anywhere.

* workers/service/ServiceWorkerJob.cpp:
(WebCore::ServiceWorkerJob::didReceiveResponse):
* workers/service/ServiceWorkerJob.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@228895 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/workers/service/ServiceWorkerJob.cpp
Source/WebCore/workers/service/ServiceWorkerJob.h

index f8b9d29..75023dd 100644 (file)
@@ -1,5 +1,18 @@
 2018-02-21  Youenn Fablet  <youenn@apple.com>
 
+        ServiceWorkerJob::m_lastResponse is unneeded
+        https://bugs.webkit.org/show_bug.cgi?id=183013
+
+        Reviewed by Chris Dumez.
+
+        Remove this field since not used anywhere.
+
+        * workers/service/ServiceWorkerJob.cpp:
+        (WebCore::ServiceWorkerJob::didReceiveResponse):
+        * workers/service/ServiceWorkerJob.h:
+
+2018-02-21  Youenn Fablet  <youenn@apple.com>
+
         Use ResourceLoader to load appcache manifest
         https://bugs.webkit.org/show_bug.cgi?id=182861
 
index 631c8c3..7606985 100644 (file)
@@ -107,7 +107,6 @@ void ServiceWorkerJob::didReceiveResponse(unsigned long, const ResourceResponse&
     ASSERT(!m_completed);
     ASSERT(m_scriptLoader);
 
-    m_lastResponse = response;
     // Extract a MIME type from the response's header list. If this MIME type (ignoring parameters) is not a JavaScript MIME type, then:
     if (!MIMETypeRegistry::isSupportedJavaScriptMIMEType(response.mimeType())) {
         // Invoke Reject Job Promise with job and "SecurityError" DOMException.
index 8471172..ee5daaf 100644 (file)
@@ -89,7 +89,6 @@ private:
 
     DocumentOrWorkerIdentifier m_contextIdentifier;
     RefPtr<WorkerScriptLoader> m_scriptLoader;
-    ResourceResponse m_lastResponse;
 
 #if !ASSERT_DISABLED
     Ref<Thread> m_creationThread { Thread::current() };