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

Unified Diff: content/browser/devtools/render_frame_devtools_agent_host.cc

Issue 988003002: DevTools: pick sw targets to attach to on the backend side. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebaselined Created 5 years, 9 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/browser/devtools/render_frame_devtools_agent_host.cc
diff --git a/content/browser/devtools/render_frame_devtools_agent_host.cc b/content/browser/devtools/render_frame_devtools_agent_host.cc
index 7f6b94241f04351451e7deb56e2677df7efff03d..81cbe7e9b4ae0ef0be01774ef0afe476e02cb4a3 100644
--- a/content/browser/devtools/render_frame_devtools_agent_host.cc
+++ b/content/browser/devtools/render_frame_devtools_agent_host.cc
@@ -159,6 +159,7 @@ RenderFrameDevToolsAgentHost::RenderFrameDevToolsAgentHost(RenderFrameHost* rfh)
dispatcher->SetServiceWorkerHandler(service_worker_handler_.get());
dispatcher->SetTracingHandler(tracing_handler_.get());
SetRenderFrameHost(rfh);
+ service_worker_handler_->SetURL(rfh->GetLastCommittedURL());
g_instances.Get().push_back(this);
AddRef(); // Balanced in RenderFrameHostDestroyed.
DevToolsManager::GetInstance()->AgentHostChanged(this);
@@ -405,6 +406,14 @@ void RenderFrameDevToolsAgentHost::NavigationEntryCommitted(
DevToolsManager::GetInstance()->AgentHostChanged(this);
}
+void RenderFrameDevToolsAgentHost::DidCommitProvisionalLoadForFrame(
+ RenderFrameHost* render_frame_host,
+ const GURL& url,
+ ui::PageTransition transition_type) {
+ if (render_frame_host_ == render_frame_host)
+ service_worker_handler_->SetURL(url);
+}
+
void RenderFrameDevToolsAgentHost::Observe(int type,
const NotificationSource& source,
const NotificationDetails& details) {

Powered by Google App Engine
This is Rietveld 408576698