MD user manager (C++ for create profile flow)

chrome/browser/ui/webui/profile_helper.h

 // Copyright 2013 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.
 
-#ifndef CHROME_BROWSER_UI_WEBUI_OPTIONS_OPTIONS_HANDLERS_HELPER_H_
-#define CHROME_BROWSER_UI_WEBUI_OPTIONS_OPTIONS_HANDLERS_HELPER_H_
+#ifndef CHROME_BROWSER_UI_WEBUI_PROFILE_HELPER_H_
+#define CHROME_BROWSER_UI_WEBUI_PROFILE_HELPER_H_
 
 #include "base/files/file_path.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/host_desktop.h"
 
 namespace content {
 class WebUI;
 }
 
-namespace options {
+namespace profiles {

[Dan Beam, 2016/02/29 20:25:32]

nit: i don't really get the "helper" namespace (in its previous form), and
profiles:: probably should be used from a /profiles/ directory (i.e.
chrome/browser/profiles)

could these just be ::webui::{OpenNewWindowForProfile,DeleteProfileAtPath}
instead?

[Moe, 2016/03/01 18:29:53]

Done.

 namespace helper {
 
 void OpenNewWindowForProfile(Profile* profile, Profile::CreateStatus status);
 
 // Deletes the profile at the given |file_path|.
 void DeleteProfileAtPath(base::FilePath file_path, content::WebUI* web_ui);
 
 }  // namespace helper
-}  // namespace options
+}  // namespace signin

[Dan Beam, 2016/02/29 20:25:33]

nit: signin -> profiles

[Moe, 2016/03/01 18:29:53]

Done.

 
 
 
-#endif  // CHROME_BROWSER_UI_WEBUI_OPTIONS_OPTIONS_HANDLERS_HELPER_H_
+#endif  // CHROME_BROWSER_UI_WEBUI_PROFILE_HELPER_H_