Implemented ForceMaximizeBrowserWindowOnFirstRun policy, added unit test and browser test.

chrome/browser/ui/ash/chrome_shell_delegate.cc

 // Copyright (c) 2012 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/ui/ash/chrome_shell_delegate.h"
 
 #include "ash/content_support/gpu_support_impl.h"
 #include "ash/wm/window_state.h"
 #include "ash/wm/window_util.h"
+#include "base/prefs/pref_service.h"
 #include "chrome/browser/app_mode/app_mode_utils.h"
 #include "chrome/browser/lifetime/application_lifetime.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/profiles/profiles_state.h"
 #include "chrome/browser/ui/app_list/app_list_view_delegate.h"
 #include "chrome/browser/ui/ash/app_list/app_list_service_ash.h"
 #include "chrome/browser/ui/ash/ash_keyboard_controller_proxy.h"
 #include "chrome/browser/ui/ash/launcher/chrome_launcher_controller.h"
 #include "chrome/browser/ui/ash/launcher/launcher_context_menu.h"
 #include "chrome/browser/ui/browser_commands.h"
+#include "chrome/common/pref_names.h"
 #include "chrome/grit/chromium_strings.h"
 #include "components/signin/core/common/profile_management_switches.h"
 #include "ui/base/l10n/l10n_util.h"
 
 #if defined(OS_CHROMEOS)
 #include "chrome/browser/chromeos/accessibility/accessibility_manager.h"
 #include "chrome/browser/chromeos/display/display_configuration_observer.h"
 #include "components/user_manager/user_manager.h"
 #endif
 
@@ skipping 51 matching lines @@
   return chrome::IsRunningInForcedAppMode();
 }
 
 bool ChromeShellDelegate::IsMultiAccountEnabled() const {
 #if defined(OS_CHROMEOS)
   return switches::IsEnableAccountConsistency();
 #endif
   return false;
 }
 
+bool ChromeShellDelegate::IsFirstTimeMaximized() const {
+#if defined(OS_CHROMEOS)
+  if (ProfileManager::GetActiveUserProfile()) {

[bartfab (slow), 2015/03/12 11:54:47]

The documentation for this method says that you should "avoid using it at all
cost" :). It is also states that for guest, you will get a "suitable" profile,
whatever that may mean. I think a better implementation is:

const user_manager::User* const user =
    user_manager::UserManager::Get()->GetActiveUser();
if (!user)
  return false;
return chromeos::ProfileHelper::Get()->GetProfileByUser(user)->
    GetPrefs()->GetBoolean(prefs::kForceMaximizeBrowserWindowOnFirstRun);

[peletskyi, 2015/03/18 13:28:13]

Done.

+    return ProfileManager::GetActiveUserProfile()->GetPrefs()->GetBoolean(

[bartfab (slow), 2015/03/12 11:54:47]

Nit: #include "chrome/browser/profiles/profile.h"

[peletskyi, 2015/03/18 13:28:13]

Done.

+        prefs::kForceMaximizeBrowserWindowOnFirstRun);
+  }
+#endif
+  return false;
+}
+
 void ChromeShellDelegate::Exit() {
   chrome::AttemptUserExit();
 }
 
 content::BrowserContext* ChromeShellDelegate::GetActiveBrowserContext() {
 #if defined(OS_CHROMEOS)
   DCHECK(user_manager::UserManager::Get()->GetLoggedInUsers().size());
 #endif
   return ProfileManager::GetActiveUserProfile();
 }
@@ skipping 51 matching lines @@
 
 void ChromeShellDelegate::AddVirtualKeyboardStateObserver(
     ash::VirtualKeyboardStateObserver* observer) {
   keyboard_state_observer_list_.AddObserver(observer);
 }
 
 void ChromeShellDelegate::RemoveVirtualKeyboardStateObserver(
     ash::VirtualKeyboardStateObserver* observer) {
   keyboard_state_observer_list_.RemoveObserver(observer);
 }