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

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: 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 f30286da0e66eaee9a7a9f14f9853b3b9adf0606..b70002629781708a4476e6bc697d30aed6347acd 100644
--- a/content/browser/devtools/render_frame_devtools_agent_host.cc
+++ b/content/browser/devtools/render_frame_devtools_agent_host.cc
@@ -151,6 +151,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);
@@ -397,6 +398,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->GetParent())
dgozman 2015/03/08 15:27:08 To account for OOPIF: if (render_frame_host_ == re
pfeldman 2015/03/08 15:34:14 Done.
+ 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