PS - Scrub URL down to origin in Public Sessions

chrome/browser/chromeos/extensions/extension_tab_util_delegate_chromeos.cc

+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/chromeos/extensions/extension_tab_util_delegate_chromeos.h"
+
+#include <string>
+
+#include "chrome/browser/profiles/profiles_state.h"
+#include "chrome/common/extensions/api/tabs.h"
+
+namespace extensions {
+
+ExtensionTabUtilDelegateChromeOS::ExtensionTabUtilDelegateChromeOS() {}
+
+ExtensionTabUtilDelegateChromeOS::~ExtensionTabUtilDelegateChromeOS() {}
+
+void ExtensionTabUtilDelegateChromeOS::ScrubTabForExtension(
+    const Extension* extension,
+    content::WebContents* contents,
+    api::tabs::Tab* tab) {
+  if (!profiles::IsPublicSession() || !tab->url) {
+    return;
+  }
+  // Scrub URL down to the origin (security reasons inside Public Sessions).
+  GURL gurl(*tab->url);
+  std::string scrubbed_url = gurl.GetOrigin().spec();
+  tab->url.reset(new std::string(scrubbed_url));

[Devlin, 2017/04/20 16:33:18]

could even one-line this:
tab->url = base::MakeUnique<std::string>(GURL(*tab->url).GetOrigin().spec());

If you think that's a little too hard to read, we should at least use
`base::MakeUnique<std::string>(std::move(scrubbed_url))` to avoid bare calls to
new and creating an unnecessary copy.

[Ivan Šandrk, 2017/04/20 16:44:10]

Roger. I did it this way because that's how it was done inside
extension_tab_util.cc

+}
+
+}  // namespace extensions