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

Unified Diff: chrome/browser/extensions/extension_service.cc

Issue 12319114: Extract debugger target enumeration into a separate class (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@debugger
Patch Set: Addressed comments Created 7 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: chrome/browser/extensions/extension_service.cc
diff --git a/chrome/browser/extensions/extension_service.cc b/chrome/browser/extensions/extension_service.cc
index 2c2156b00c2cea9df5faba33b993711eb96ae9ed..688e75279fcbdd1d93bbdd00765706d2b8cf406b 100644
--- a/chrome/browser/extensions/extension_service.cc
+++ b/chrome/browser/extensions/extension_service.cc
@@ -733,9 +733,9 @@ void ExtensionService::ReloadExtensionWithEvents(
manager->GetBackgroundHostForExtension(extension_id);
if (host && DevToolsAgentHost::HasFor(host->render_view_host())) {
// Look for an open inspector for the background page.
- int devtools_cookie = DevToolsAgentHost::DisconnectRenderViewHost(
+ std::string devtools_cookie = DevToolsAgentHost::DisconnectRenderViewHost(
host->render_view_host());
- if (devtools_cookie >= 0)
+ if (devtools_cookie != DevToolsAgentHost::NullId)
orphaned_dev_tools_[extension_id] = devtools_cookie;
}

Powered by Google App Engine
This is Rietveld 408576698