Index: third_party/WebKit/Source/core/loader/WorkerFetchContext.cpp |
diff --git a/third_party/WebKit/Source/core/loader/WorkerFetchContext.cpp b/third_party/WebKit/Source/core/loader/WorkerFetchContext.cpp |
index 9f21b6665a3165a2db2e9d5eefa2935a9790a0b4..518bc890877eba590148156c8318d102716f4158 100644 |
--- a/third_party/WebKit/Source/core/loader/WorkerFetchContext.cpp |
+++ b/third_party/WebKit/Source/core/loader/WorkerFetchContext.cpp |
@@ -116,7 +116,10 @@ SubresourceFilter* WorkerFetchContext::GetSubresourceFilter() const { |
} |
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; |
} |
@@ -134,11 +137,14 @@ void WorkerFetchContext::DispatchDidBlockRequest( |
} |
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; |
} |