Removing pre-material design menu setting.

chrome/browser/sync/sync_global_error_unittest.cc

Index: chrome/browser/sync/sync_global_error_unittest.cc
diff --git a/chrome/browser/sync/sync_global_error_unittest.cc b/chrome/browser/sync/sync_global_error_unittest.cc
deleted file mode 100644
index be3aa6ac01bf41c76a1ebb12f7162dfc8a8f01ee..0000000000000000000000000000000000000000
--- a/chrome/browser/sync/sync_global_error_unittest.cc
+++ /dev/null
@@ -1,178 +0,0 @@
-// 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/sync/sync_global_error.h"
-
-#include <memory>
-
-#include "base/memory/ptr_util.h"
-#include "chrome/browser/chrome_notification_types.h"
-#include "chrome/browser/sync/profile_sync_test_util.h"
-#include "chrome/browser/sync/sync_global_error_factory.h"
-#include "chrome/browser/ui/browser.h"
-#include "chrome/browser/ui/global_error/global_error_service_factory.h"
-#include "chrome/browser/ui/webui/signin/login_ui_service.h"
-#include "chrome/browser/ui/webui/signin/login_ui_service_factory.h"
-#include "chrome/test/base/browser_with_test_window_test.h"
-#include "chrome/test/base/testing_profile.h"
-#include "components/signin/core/common/profile_management_switches.h"
-#include "content/public/browser/notification_observer.h"
-#include "content/public/browser/notification_registrar.h"
-#include "content/public/browser/notification_source.h"
-#include "testing/gmock/include/gmock/gmock.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-using ::testing::NiceMock;
-using ::testing::Return;
-using ::testing::ReturnRef;
-using ::testing::_;
-
-namespace {
-
-class FakeLoginUIService: public LoginUIService {
- public:
-  FakeLoginUIService() : LoginUIService(nullptr) {}
-};
-
-class FakeLoginUI : public LoginUIService::LoginUI {
- public:
-  FakeLoginUI() : focus_ui_call_count_(0) {}
-
-  ~FakeLoginUI() override {}
-
-  int focus_ui_call_count() const { return focus_ui_call_count_; }
-
- private:
-  // Overridden from LoginUIService::LoginUI:
-  void FocusUI() override { ++focus_ui_call_count_; }
-
-  int focus_ui_call_count_;
-};
-
-std::unique_ptr<KeyedService> BuildMockLoginUIService(
-    content::BrowserContext* profile) {
-  return base::MakeUnique<FakeLoginUIService>();
-}
-
-// Same as BrowserWithTestWindowTest, but uses MockBrowser to test calls to
-// ExecuteCommand method.
-class SyncGlobalErrorTest : public BrowserWithTestWindowTest {
- public:
-  SyncGlobalErrorTest() {}
-  ~SyncGlobalErrorTest() override {}
-
-  void SetUp() override {
-    profile_ = MakeSignedInTestingProfile();
-
-    BrowserWithTestWindowTest::SetUp();
-  }
-
-  Profile* profile() { return profile_.get(); }
-
- private:
-  std::unique_ptr<TestingProfile> profile_;
-
-  DISALLOW_COPY_AND_ASSIGN(SyncGlobalErrorTest);
-};
-
-// Utility function to test that SyncGlobalError behaves correctly for the given
-// error condition.
-void VerifySyncGlobalErrorResult(browser_sync::ProfileSyncServiceMock* service,
-                                 FakeLoginUIService* login_ui_service,
-                                 Browser* browser,
-                                 syncer::SyncErrorController* error,
-                                 SyncGlobalError* global_error,
-                                 GoogleServiceAuthError::State error_state,
-                                 bool is_signed_in,
-                                 bool is_error) {
-  EXPECT_CALL(*service, IsFirstSetupComplete())
-      .WillRepeatedly(Return(is_signed_in));
-
-  GoogleServiceAuthError auth_error(error_state);
-  EXPECT_CALL(*service, GetAuthError()).WillRepeatedly(ReturnRef(auth_error));
-
-  error->OnStateChanged(service);
-  EXPECT_EQ(is_error, error->HasError());
-
-  // If there is an error then a menu item and bubble view should be shown.
-  EXPECT_EQ(is_error, global_error->HasMenuItem());
-  EXPECT_EQ(is_error, global_error->HasBubbleView());
-
-  // If there is an error then labels should not be empty.
-  EXPECT_NE(0, global_error->MenuItemCommandID());
-  EXPECT_NE(is_error, global_error->MenuItemLabel().empty());
-  EXPECT_NE(is_error, global_error->GetBubbleViewAcceptButtonLabel().empty());
-
-  // We never have a cancel button.
-  EXPECT_TRUE(global_error->GetBubbleViewCancelButtonLabel().empty());
-  // We always return a hardcoded title.
-  EXPECT_FALSE(global_error->GetBubbleViewTitle().empty());
-
-  // Test message handler.
-  if (is_error) {
-    FakeLoginUI* login_ui = static_cast<FakeLoginUI*>(
-        login_ui_service->current_login_ui());
-    global_error->ExecuteMenuItem(browser);
-    ASSERT_GT(login_ui->focus_ui_call_count(), 0);
-    global_error->BubbleViewAcceptButtonPressed(browser);
-    global_error->BubbleViewDidClose(browser);
-  }
-}
-
-}  // namespace
-
-// Test that SyncGlobalError shows an error if a passphrase is required.
-TEST_F(SyncGlobalErrorTest, PassphraseGlobalError) {
-  // The MD User Menu displays Sync errors in a different way and should be the
-  // only one to do so. Under that paradigm, this test is obsolete.
-  if (switches::IsMaterialDesignUserMenu())
-    return;
-
-  browser_sync::ProfileSyncServiceMock service(
-      CreateProfileSyncServiceParamsForTest(profile()));
-
-  FakeLoginUIService* login_ui_service = static_cast<FakeLoginUIService*>(
-      LoginUIServiceFactory::GetInstance()->SetTestingFactoryAndUse(
-          profile(), BuildMockLoginUIService));
-  FakeLoginUI login_ui;
-  login_ui_service->SetLoginUI(&login_ui);
-
-  syncer::SyncErrorController error(&service);
-  SyncGlobalError global_error(
-      GlobalErrorServiceFactory::GetForProfile(profile()), login_ui_service,
-      &error, &service);
-
-  syncer::SyncEngine::Status status;
-  EXPECT_CALL(service, QueryDetailedSyncStatus(_))
-              .WillRepeatedly(Return(false));
-
-  EXPECT_CALL(service, IsPassphraseRequired())
-              .WillRepeatedly(Return(true));
-  EXPECT_CALL(service, IsPassphraseRequiredForDecryption())
-              .WillRepeatedly(Return(true));
-  VerifySyncGlobalErrorResult(
-      &service, login_ui_service, browser(), &error, &global_error,
-      GoogleServiceAuthError::NONE, true /* signed in*/, true /* error */);
-
-  // Check that no menu item is shown if there is no error.
-  EXPECT_CALL(service, IsPassphraseRequired())
-              .WillRepeatedly(Return(false));
-  EXPECT_CALL(service, IsPassphraseRequiredForDecryption())
-              .WillRepeatedly(Return(false));
-  VerifySyncGlobalErrorResult(
-      &service, login_ui_service, browser(), &error, &global_error,
-      GoogleServiceAuthError::NONE, true /* signed in */, false /* no error */);
-
-  // Check that no menu item is shown if sync setup is not completed.
-  EXPECT_CALL(service, IsPassphraseRequired())
-              .WillRepeatedly(Return(true));
-  EXPECT_CALL(service, IsPassphraseRequiredForDecryption())
-              .WillRepeatedly(Return(true));
-  VerifySyncGlobalErrorResult(
-      &service, login_ui_service, browser(), &error, &global_error,
-      GoogleServiceAuthError::INVALID_GAIA_CREDENTIALS,
-      false /* not signed in */, false /* no error */);
-
-  global_error.Shutdown();
-}