Use mojo SystemTray interfaces for both mash and classic ash

chrome/browser/ui/ash/system_tray_delegate_chromeos_browsertest_chromeos.cc

 // Copyright 2014 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/system_tray_delegate_chromeos.h"
 
 #include <string>
 
 #include "ash/common/system/date/date_default_view.h"
 #include "ash/common/system/date/date_view.h"
@@ skipping 73 matching lines @@
         account_id1_(AccountId::FromUserEmail(kUser1)),
         account_id2_(AccountId::FromUserEmail(kUser2)) {}
 
   ~SystemTrayDelegateChromeOSTest() override {}
 
   void SetupUserProfile(const AccountId& account_id, bool use_24_hour_clock) {
     const user_manager::User* user =
         user_manager::UserManager::Get()->FindUser(account_id);
     Profile* profile = ProfileHelper::Get()->GetProfileByUser(user);
     profile->GetPrefs()->SetBoolean(prefs::kUse24HourClock, use_24_hour_clock);
+    // Allow clock setting to be sent to ash over mojo.
+    content::RunAllPendingInMessageLoop();
   }
 
   const AccountId account_id1_;
   const AccountId account_id2_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(SystemTrayDelegateChromeOSTest);
 };
 
 IN_PROC_BROWSER_TEST_F(SystemTrayDelegateChromeOSTest,
                        PRE_TestMultiProfile24HourClock) {
   RegisterUser(account_id1_.GetUserEmail());
   RegisterUser(account_id2_.GetUserEmail());
   StartupUtils::MarkOobeCompleted();
 }
 
 // Test that clock type is taken from user profile for current active user.
 IN_PROC_BROWSER_TEST_F(SystemTrayDelegateChromeOSTest,
                        TestMultiProfile24HourClock) {
   LoginUser(account_id1_.GetUserEmail());
   SetupUserProfile(account_id1_, true /* Use_24_hour_clock. */);
   CreateDefaultView();
   EXPECT_EQ(base::k24HourClock, GetHourType());
+
   UserAddingScreen::Get()->Start();
   content::RunAllPendingInMessageLoop();
   AddUser(account_id2_.GetUserEmail());
   SetupUserProfile(account_id2_, false /* Use_24_hour_clock. */);
   CreateDefaultView();
   EXPECT_EQ(base::k12HourClock, GetHourType());
+
   user_manager::UserManager::Get()->SwitchActiveUser(account_id1_);
+  // Allow clock setting to be sent to ash over mojo.
+  content::RunAllPendingInMessageLoop();
   CreateDefaultView();
   EXPECT_EQ(base::k24HourClock, GetHourType());
 }
 
 // Makes sure that no notifications are shown when rotating the
 // display on display settings URLs.
 IN_PROC_BROWSER_TEST_F(DisplayNotificationsTest,
                        TestDisplayOrientationChangeNotification) {
   // Open the display settings page.
   ui_test_utils::NavigateToURL(browser(),
@@ skipping 19 matching lines @@
 
   UpdateDisplay("400x400");
 
   ui_test_utils::NavigateToURL(browser(), GURL("chrome://version"));
   UpdateDisplay("400x400/r");
   // Ensure that there is a notification that is shown.
   EXPECT_FALSE(GetVisibleNotifications().empty());
 }
 
 }  // namespace chromeos