Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1401)

Unified Diff: content/renderer/service_worker/worker_fetch_context_impl.cc

Issue 2888333002: Send certificate errors from worker fetch context for off-main-thread-fetch. (Closed)
Patch Set: incorporated kinuko's comment Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/service_worker/worker_fetch_context_impl.cc
diff --git a/content/renderer/service_worker/worker_fetch_context_impl.cc b/content/renderer/service_worker/worker_fetch_context_impl.cc
index e8180176d55fc93ff62ceab6eb66a65223577ef8..a2c6d70316f06a9c233052fe3c9076bd45fca003 100644
--- a/content/renderer/service_worker/worker_fetch_context_impl.cc
+++ b/content/renderer/service_worker/worker_fetch_context_impl.cc
@@ -4,16 +4,20 @@
#include "content/renderer/service_worker/worker_fetch_context_impl.h"
+#include "content/child/child_thread_impl.h"
#include "content/child/request_extra_data.h"
#include "content/child/resource_dispatcher.h"
+#include "content/child/thread_safe_sender.h"
#include "content/child/web_url_loader_impl.h"
+#include "content/common/frame_messages.h"
#include "mojo/public/cpp/bindings/associated_binding.h"
namespace content {
WorkerFetchContextImpl::WorkerFetchContextImpl(
mojom::WorkerURLLoaderFactoryProviderPtrInfo provider_info)
- : provider_info_(std::move(provider_info)) {}
+ : provider_info_(std::move(provider_info)),
+ thread_safe_sender_(ChildThreadImpl::current()->thread_safe_sender()) {}
WorkerFetchContextImpl::~WorkerFetchContextImpl() {}
@@ -71,6 +75,18 @@ blink::WebURL WorkerFetchContextImpl::FirstPartyForCookies() const {
return first_party_for_cookies_;
}
+void WorkerFetchContextImpl::DidRunContentWithCertificateErrors(
+ const blink::WebURL& url) {
+ Send(new FrameHostMsg_DidRunContentWithCertificateErrors(parent_frame_id_,
+ url));
+}
+
+void WorkerFetchContextImpl::DidDisplayContentWithCertificateErrors(
+ const blink::WebURL& url) {
+ Send(new FrameHostMsg_DidDisplayContentWithCertificateErrors(parent_frame_id_,
+ url));
+}
+
void WorkerFetchContextImpl::set_service_worker_provider_id(int id) {
service_worker_provider_id_ = id;
}
@@ -93,4 +109,8 @@ void WorkerFetchContextImpl::SetControllerServiceWorker(
controller_version_id_ = controller_version_id;
}
+bool WorkerFetchContextImpl::Send(IPC::Message* message) {
+ return thread_safe_sender_->Send(message);
+}
+
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698