Remove unsuitable TODO comments in WorkerFetchContext

third_party/WebKit/Source/core/loader/WorkerFetchContext.cpp

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/loader/WorkerFetchContext.h"
 
 #include "core/frame/Deprecation.h"
 #include "core/frame/UseCounter.h"
 #include "core/loader/MixedContentChecker.h"
 #include "core/timing/WorkerGlobalScopePerformance.h"
@@ skipping 98 matching lines @@
   // TODO(horo): Implement this.
   return nullptr;
 }
 
 SubresourceFilter* WorkerFetchContext::GetSubresourceFilter() const {
   // TODO(horo): Implement this.
   return nullptr;
 }
 
 SecurityContext* WorkerFetchContext::GetParentSecurityContext() const {
-  // TODO(horo): Implement this.
+  // This method was introduced to check the parent frame's security context
+  // while loading iframe document resources. So this method is not suitable for
+  // workers.
+  NOTREACHED();
   return nullptr;
 }
 
 bool WorkerFetchContext::ShouldBlockRequestByInspector(
     const ResourceRequest& resource_request) const {
   // TODO(horo): Implement this.
   return false;
 }
 
 void WorkerFetchContext::DispatchDidBlockRequest(
     const ResourceRequest& resource_request,
     const FetchInitiatorInfo& fetch_initiator_info,
     ResourceRequestBlockedReason blocked_reason) const {
   // TODO(horo): Implement this.
 }
 
 void WorkerFetchContext::ReportLocalLoadFailed(const KURL&) const {
-  // TODO(horo): Implement this.
+  // Threre is no way to load local files from worker thread.
+  NOTREACHED();
 }
 
 bool WorkerFetchContext::ShouldBypassMainWorldCSP() const {
-  // TODO(horo): Implement this.
+  // This method was introduced to bypass the page's CSP while running the
+  // script from an isolated world (ex: Chrome extensions). But worker threads
+  // doesn't have any isolated world. So we can just return false.
   return false;
 }
 
 bool WorkerFetchContext::IsSVGImageChromeClient() const {
   return false;
 }
 
 void WorkerFetchContext::CountUsage(UseCounter::Feature feature) const {
   UseCounter::Count(worker_global_scope_, feature);
 }
@@ skipping 98 matching lines @@
 void ProvideWorkerFetchContextToWorker(
     WorkerClients* clients,
     std::unique_ptr<WebWorkerFetchContext> web_context) {
   DCHECK(clients);
   WorkerFetchContextHolder::ProvideTo(
       *clients, WorkerFetchContextHolder::SupplementName(),
       new WorkerFetchContextHolder(std::move(web_context)));
 }
 
 }  // namespace blink