Prevent new Browser windows from violating the IncognitoModeAvailability policy.

chrome/browser/prefs/incognito_mode_prefs.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/prefs/incognito_mode_prefs.h"
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/prefs/pref_service.h"
+#include "chrome/browser/profiles/profile.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/pref_names.h"
 #include "components/user_prefs/pref_registry_syncable.h"
 
 #if defined(OS_WIN)
 #include "base/win/metro.h"
 #endif  // OS_WIN
 
 // static
 bool IncognitoModePrefs::IntToAvailability(int in_value,
@@ skipping 43 matching lines @@
 
 // static
 bool IncognitoModePrefs::ShouldLaunchIncognito(
     const CommandLine& command_line,
     const PrefService* prefs) {
   Availability incognito_avail = GetAvailability(prefs);
   return incognito_avail != IncognitoModePrefs::DISABLED &&
          (command_line.HasSwitch(switches::kIncognito) ||
           incognito_avail == IncognitoModePrefs::FORCED);
 }
+
+// static
+bool IncognitoModePrefs::CanOpenBrowser(Profile* profile) {
+  // If this causes a crash in Browser then a window is being opened using a
+  // profile type that is disallowed by policy. The crash prevents the
+  // "forbidden" window type from opening at all, but the path that triggered
+  // it should be fixed.

[Mattias Nissler (ping if slow), 2013/05/29 11:33:08]

Shouldn't this comment go with the CHECK()?

[Joao da Silva, 2013/05/29 11:40:40]

Done.

+  switch (GetAvailability(profile->GetPrefs())) {
+    case IncognitoModePrefs::ENABLED:
+      return true;
+    case IncognitoModePrefs::DISABLED:
+      return !profile->IsOffTheRecord();
+    case IncognitoModePrefs::FORCED:
+      return profile->IsOffTheRecord();
+    case IncognitoModePrefs::AVAILABILITY_NUM_TYPES:
+      NOTREACHED();
+  }
+  NOTREACHED();
+  return false;
+}