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

Unified Diff: chrome/browser/external_protocol/external_protocol_handler.cc

Issue 1657933003: Fixes the interactive default browser UX for policy setting (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: grt comments Created 4 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: chrome/browser/external_protocol/external_protocol_handler.cc
diff --git a/chrome/browser/external_protocol/external_protocol_handler.cc b/chrome/browser/external_protocol/external_protocol_handler.cc
index 90b54281b32cdaca44d9dacc26c84a815fd45d65..978cbe200bf8685f091f40f5cc444fac8a49c336 100644
--- a/chrome/browser/external_protocol/external_protocol_handler.cc
+++ b/chrome/browser/external_protocol/external_protocol_handler.cc
@@ -44,8 +44,8 @@ ShellIntegration::DefaultProtocolClientWorker* CreateShellWorker(
const std::string& protocol,
ExternalProtocolHandler::Delegate* delegate) {
if (!delegate)
- return new ShellIntegration::DefaultProtocolClientWorker(observer,
- protocol);
+ return new ShellIntegration::DefaultProtocolClientWorker(observer, protocol,
+ true);
return delegate->CreateShellWorker(observer, protocol);
}
@@ -146,8 +146,6 @@ class ExternalDefaultProtocolObserver
escaped_url_, render_process_host_id_, tab_contents_id_, delegate_);
}
- bool IsOwnedByWorker() override { return true; }
-
private:
ExternalProtocolHandler::Delegate* delegate_;
const GURL escaped_url_;

Powered by Google App Engine
This is Rietveld 408576698