PS - Scrub URL down to origin in Public Sessions

chrome/browser/chromeos/login/users/chrome_user_manager_impl.cc

Index: chrome/browser/chromeos/login/users/chrome_user_manager_impl.cc
diff --git a/chrome/browser/chromeos/login/users/chrome_user_manager_impl.cc b/chrome/browser/chromeos/login/users/chrome_user_manager_impl.cc
index 4411a592dd6c5bfbaea828fb19d7eb592b948a62..f4f71f8fa70a26db4f7c4059071fe94120fa982a 100644
--- a/chrome/browser/chromeos/login/users/chrome_user_manager_impl.cc
+++ b/chrome/browser/chromeos/login/users/chrome_user_manager_impl.cc
@@ -32,6 +32,7 @@
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/chromeos/app_mode/kiosk_app_manager.h"
+#include "chrome/browser/chromeos/extensions/extension_tab_util_delegate_chromeos.h"
 #include "chrome/browser/chromeos/extensions/permissions_updater_delegate_chromeos.h"
 #include "chrome/browser/chromeos/login/demo_mode/demo_app_launcher.h"
 #include "chrome/browser/chromeos/login/enterprise_user_session_metrics.h"
@@ -52,6 +53,7 @@
 #include "chrome/browser/chromeos/settings/cros_settings.h"
 #include "chrome/browser/chromeos/system/timezone_resolver_manager.h"
 #include "chrome/browser/chromeos/system/timezone_util.h"
+#include "chrome/browser/extensions/extension_tab_util.h"
 #include "chrome/browser/extensions/permissions_updater.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/signin/easy_unlock_service.h"
@@ -842,6 +844,11 @@ void ChromeUserManagerImpl::PublicAccountUserLoggedIn(
   // logout), ie. it's not freed and it leaks but that is fine.
   extensions::PermissionsUpdater::SetPlatformDelegate(
       new extensions::PermissionsUpdaterDelegateChromeOS);
+
+  // In Public Sessions set the PS delegate on ExtensionTabUtil (used to scrub

[Ivan Šandrk, 2017/04/20 16:14:47]

Should I make this comment longer perhaps? I was thinking that this is maybe
fine since there's a similar comment right before it.

[Alexander Alekseev, 2017/04/24 11:35:53]

I think we should reference PermissionsUpdaterDelegateChromeOS here just in case
something is inserted in between in the future.

Could you add a comment like '(like PermissionsUpdaterDelegateChromeOS above)' ?

[Ivan Šandrk, 2017/04/24 12:14:56]

Done.

+  // URL down to origin for security reasons).
+  extensions::ExtensionTabUtil::SetPlatformDelegate(
+      new extensions::ExtensionTabUtilDelegateChromeOS);
 }
 
 void ChromeUserManagerImpl::KioskAppLoggedIn(user_manager::User* user) {