Index: chrome/renderer/extensions/chrome_extensions_dispatcher_delegate.cc |
diff --git a/chrome/renderer/extensions/chrome_extensions_dispatcher_delegate.cc b/chrome/renderer/extensions/chrome_extensions_dispatcher_delegate.cc |
index 8640d30fc66354ea1421f660fbca6f179bad5efb..b8576957a273f8e73fb910ec507ebe2fc8597eca 100644 |
--- a/chrome/renderer/extensions/chrome_extensions_dispatcher_delegate.cc |
+++ b/chrome/renderer/extensions/chrome_extensions_dispatcher_delegate.cc |
@@ -307,11 +307,12 @@ void ChromeExtensionsDispatcherDelegate::UpdateTabSpecificPermissions( |
const std::string& extension_id, |
const extensions::URLPatternSet& origin_set) { |
content::RenderView* view = extensions::TabFinder::Find(tab_id); |
+ if (!view) |
+ return; |
// For now, the message should only be sent to the render view that contains |
not at google - send to devlin
2014/11/12 16:28:09
Was this actually crashing before? The comment ind
msimonides
2014/11/13 17:13:20
Unfortunately I'm unable to reproduce the crash, w
not at google - send to devlin
2014/11/13 18:17:37
Good question. I'm seeing crashes on line 312, GUR
|
// the target tab. This may change. Either way, if this is the target tab it |
// gives us the chance to check against the URL to avoid races. |
- DCHECK(view); |
GURL active_url(view->GetWebView()->mainFrame()->document().url()); |
if (active_url != url) |
return; |