Use shark requisition instead of command line parameter

chrome/browser/chromeos/login/wizard_controller.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/chromeos/login/wizard_controller.h"
 
 #include <signal.h>
 #include <stdlib.h>
 #include <sys/types.h>
 
@@ skipping 34 matching lines @@
 #include "chrome/browser/chromeos/login/screens/update_screen.h"
 #include "chrome/browser/chromeos/login/screens/user_image_screen.h"
 #include "chrome/browser/chromeos/login/screens/wrong_hwid_screen.h"
 #include "chrome/browser/chromeos/login/startup_utils.h"
 #include "chrome/browser/chromeos/login/supervised/supervised_user_creation_screen.h"
 #include "chrome/browser/chromeos/login/ui/login_display_host.h"
 #include "chrome/browser/chromeos/login/ui/oobe_display.h"
 #include "chrome/browser/chromeos/net/delay_network_call.h"
 #include "chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h"
 #include "chrome/browser/chromeos/policy/device_cloud_policy_initializer.h"
+#include "chrome/browser/chromeos/policy/device_cloud_policy_manager_chromeos.h"
 #include "chrome/browser/chromeos/settings/cros_settings.h"
 #include "chrome/browser/chromeos/timezone/timezone_provider.h"
 #include "chrome/browser/lifetime/application_lifetime.h"
 #include "chrome/browser/metrics/metrics_reporting_state.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/pref_names.h"
 #include "chromeos/audio/cras_audio_handler.h"
@@ skipping 33 matching lines @@
   chromeos::WizardController::kTermsOfServiceScreenName,
   chromeos::WizardController::kAutoEnrollmentCheckScreenName
 };
 
 // Checks flag for HID-detection screen show.
 bool CanShowHIDDetectionScreen() {
   return !CommandLine::ForCurrentProcess()->HasSwitch(
         chromeos::switches::kDisableHIDDetectionOnOOBE);
 }
 
 bool ShouldShowControllerPairingScreen() {

[achuithb, 2014/08/29 23:03:34]

Why not get rid of this altogether?

[Zachary Kuznia, 2014/08/29 23:27:10]

Done.

-  return CommandLine::ForCurrentProcess()->HasSwitch(
-      chromeos::switches::kShowControllerPairingDemo);
+  const bool is_shark =
+      g_browser_process->platform_part()->browser_policy_connector_chromeos()->

[achuithb, 2014/08/29 23:03:34]

Might as well just return this instead of creating the temporary is_shark.

[Zachary Kuznia, 2014/08/29 23:27:10]

Function removed.

+          GetDeviceCloudPolicyManager()->IsSharkRequisition();
+
+  return is_shark;
 }
 
 bool ShouldShowHostPairingScreen() {
   return CommandLine::ForCurrentProcess()->HasSwitch(
       chromeos::switches::kShowHostPairingDemo);
 }
 
 bool IsResumableScreen(const std::string& screen) {
   for (size_t i = 0; i < ARRAYSIZE_UNSAFE(kResumableScreens); ++i) {
     if (screen == kResumableScreens[i])
@@ skipping 1173 matching lines @@
   if (ShouldAutoStartEnrollment() || ShouldRecoverEnrollment()) {
     StartupUtils::MarkEulaAccepted();
     auth_token_ = token;
     InitiateOOBEUpdate();
   } else {
     LOG(WARNING) << "Not in device enrollment.";
   }
 }
 
 }  // namespace chromeos