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

Unified Diff: content/browser/debugger/devtools_netlog_observer.cc

Issue 9113079: DevTools: refactor remote debugging server to enable content_shell debugging. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Patch for landing. Created 8 years, 11 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/debugger/devtools_netlog_observer.cc
diff --git a/content/browser/debugger/devtools_netlog_observer.cc b/content/browser/debugger/devtools_netlog_observer.cc
index 98492b2b0c76380411bd58ebf839bb124adc0193..a563a8d486dba003862574a07cf96b92d5c3a29a 100644
--- a/content/browser/debugger/devtools_netlog_observer.cc
+++ b/content/browser/debugger/devtools_netlog_observer.cc
@@ -240,17 +240,18 @@ void DevToolsNetLogObserver::OnAddSocketEntry(
void DevToolsNetLogObserver::Attach() {
DCHECK(!instance_);
-
- instance_ = new DevToolsNetLogObserver(
- content::GetContentClient()->browser()->GetNetLog());
+ net::NetLog* net_log = content::GetContentClient()->browser()->GetNetLog();
+ if (net_log)
+ instance_ = new DevToolsNetLogObserver(net_log);
}
void DevToolsNetLogObserver::Detach() {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
- DCHECK(instance_);
- delete instance_;
- instance_ = NULL;
+ if (instance_) {
+ delete instance_;
+ instance_ = NULL;
+ }
}
DevToolsNetLogObserver* DevToolsNetLogObserver::GetInstance() {

Powered by Google App Engine
This is Rietveld 408576698