Chromium Code Reviews| Index: chrome/browser/ui/browser.cc |
| diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc |
| index ac738adafae528c8c6d15f2d785af3f18df44531..f7b71f64c82c67bb1710bbf1a7065fe30e53157d 100644 |
| --- a/chrome/browser/ui/browser.cc |
| +++ b/chrome/browser/ui/browser.cc |
| @@ -1760,6 +1760,7 @@ void Browser::RegisterProtocolHandler(WebContents* web_contents, |
| PermissionBubbleManager* bubble_manager = |
|
please use gerrit instead
2015/08/05 18:05:25
Shouldn't PermissionBubbleManager go away entirely
hcarmona
2015/08/05 18:43:47
Permission bubbles need a browser to get their win
hcarmona
2015/08/06 17:43:43
From in person discussion: The delegate will be ow
|
| PermissionBubbleManager::FromWebContents(web_contents); |
| if (PermissionBubbleManager::Enabled() && bubble_manager) { |
| + bubble_manager->SetBrowser(this); |
| bubble_manager->AddRequest( |
| new RegisterProtocolHandlerPermissionRequest(registry, handler, |
| url, user_gesture)); |