Gut ash::MaterialDesignController, and remove the about:flags entry.

ash/display/display_manager_unittest.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 "ui/display/manager/display_manager.h"
 
 #include "ash/accelerators/accelerator_commands_aura.h"
 #include "ash/common/ash_switches.h"
-#include "ash/common/material_design/material_design_controller.h"
 #include "ash/common/wm/window_state.h"
 #include "ash/display/display_configuration_controller.h"
 #include "ash/display/display_util.h"
 #include "ash/display/mirror_window_controller.h"
 #include "ash/display/window_tree_host_manager.h"
 #include "ash/screen_util.h"
 #include "ash/shell.h"
-#include "ash/test/ash_md_test_base.h"
 #include "ash/test/ash_test_base.h"
 #include "ash/test/mirror_window_test_api.h"
 #include "ash/wm/window_state_aura.h"
 #include "base/command_line.h"
 #include "base/format_macros.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/stringprintf.h"
 #include "build/build_config.h"
 #include "grit/ash_strings.h"
 #include "ui/aura/client/aura_constants.h"
@@ skipping 21 matching lines @@
 using base::StringPrintf;
 
 namespace {
 
 std::string ToDisplayName(int64_t id) {
   return "x-" + base::Int64ToString(id);
 }
 
 }  // namespace
 
-class DisplayManagerTest : public test::AshMDTestBase,
+class DisplayManagerTest : public test::AshTestBase,
                            public display::DisplayObserver,
                            public aura::WindowObserver {
  public:
   DisplayManagerTest()
       : removed_count_(0U),
         root_window_destroyed_(false),
         changed_metrics_(0U) {}
   ~DisplayManagerTest() override {}
 
   void SetUp() override {
-    AshMDTestBase::SetUp();
+    AshTestBase::SetUp();
     display::Screen::GetScreen()->AddObserver(this);
     Shell::GetPrimaryRootWindow()->AddObserver(this);
   }
   void TearDown() override {
     Shell::GetPrimaryRootWindow()->RemoveObserver(this);
     display::Screen::GetScreen()->RemoveObserver(this);
-    AshMDTestBase::TearDown();
+    AshTestBase::TearDown();
   }
 
   const vector<display::Display>& changed() const { return changed_; }
   const vector<display::Display>& added() const { return added_; }
   uint32_t changed_metrics() const { return changed_metrics_; }
 
   string GetCountSummary() const {
     return StringPrintf("%" PRIuS " %" PRIuS " %" PRIuS, changed_.size(),
                         added_.size(), removed_count_);
   }
@@ skipping 47 matching lines @@
  private:
   vector<display::Display> changed_;
   vector<display::Display> added_;
   size_t removed_count_;
   bool root_window_destroyed_;
   uint32_t changed_metrics_;
 
   DISALLOW_COPY_AND_ASSIGN(DisplayManagerTest);
 };
 
-INSTANTIATE_TEST_CASE_P(
-    /* prefix intentionally left blank due to only one parameterization */,
-    DisplayManagerTest,
-    testing::Values(MaterialDesignController::NON_MATERIAL,
-                    MaterialDesignController::MATERIAL_NORMAL,
-                    MaterialDesignController::MATERIAL_EXPERIMENTAL));
-
-TEST_P(DisplayManagerTest, UpdateDisplayTest) {
+TEST_F(DisplayManagerTest, UpdateDisplayTest) {
   EXPECT_EQ(1U, display_manager()->GetNumDisplays());
 
   // Update primary and add seconary.
   UpdateDisplay("100+0-500x500,0+501-400x400");
   EXPECT_EQ(2U, display_manager()->GetNumDisplays());
   EXPECT_EQ("0,0 500x500",
             display_manager()->GetDisplayAt(0).bounds().ToString());
 
   EXPECT_EQ("2 1 0", GetCountSummary());
   EXPECT_EQ(display_manager()->GetDisplayAt(1).id(), changed()[0].id());
@@ skipping 81 matching lines @@
   // Changing primary will update secondary as well.
   UpdateDisplay("0+0-800x600,1000+1000-600x400");
   EXPECT_EQ("2 0 0", GetCountSummary());
   reset();
   EXPECT_EQ("0,0 800x600",
             display_manager()->GetDisplayAt(0).bounds().ToString());
   EXPECT_EQ("800,0 600x400",
             display_manager()->GetDisplayAt(1).bounds().ToString());
 }
 
-TEST_P(DisplayManagerTest, ScaleOnlyChange) {
+TEST_F(DisplayManagerTest, ScaleOnlyChange) {
   display_manager()->ToggleDisplayScaleFactor();
   EXPECT_TRUE(changed_metrics() &
               display::DisplayObserver::DISPLAY_METRIC_BOUNDS);
   EXPECT_TRUE(changed_metrics() &
               display::DisplayObserver::DISPLAY_METRIC_WORK_AREA);
 }
 
 // Test in emulation mode (use_fullscreen_host_window=false)
-TEST_P(DisplayManagerTest, EmulatorTest) {
+TEST_F(DisplayManagerTest, EmulatorTest) {
   EXPECT_EQ(1U, display_manager()->GetNumDisplays());
 
   display_manager()->AddRemoveDisplay();
   // Update primary and add seconary.
   EXPECT_EQ(2U, display_manager()->GetNumDisplays());
   EXPECT_EQ("1 1 0", GetCountSummary());
   reset();
 
   display_manager()->AddRemoveDisplay();
   EXPECT_EQ(1U, display_manager()->GetNumDisplays());
   EXPECT_EQ("0 0 1", GetCountSummary());
   reset();
 
   display_manager()->AddRemoveDisplay();
   EXPECT_EQ(2U, display_manager()->GetNumDisplays());
   EXPECT_EQ("1 1 0", GetCountSummary());
 }
 
 // Tests support for 3 displays.
-TEST_P(DisplayManagerTest, UpdateThreeDisplaysWithDefaultLayout) {
+TEST_F(DisplayManagerTest, UpdateThreeDisplaysWithDefaultLayout) {
   EXPECT_EQ(1U, display_manager()->GetNumDisplays());
 
   // Test with three displays. Native origin will not affect ash
   // display layout.
   UpdateDisplay("0+0-640x480,1000+0-320x200,2000+0-400x300");
 
   EXPECT_EQ(3U, display_manager()->GetNumDisplays());
   EXPECT_EQ("0,0 640x480",
             display_manager()->GetDisplayAt(0).bounds().ToString());
   EXPECT_EQ("640,0 320x200",
@@ skipping 38 matching lines @@
   UpdateDisplay("640x480,320x200,400x300");
 
   EXPECT_EQ("0,0 640x480",
             display_manager()->GetDisplayAt(0).bounds().ToString());
   EXPECT_EQ("10,480 320x200",
             display_manager()->GetDisplayAt(1).bounds().ToString());
   EXPECT_EQ("20,680 400x300",
             display_manager()->GetDisplayAt(2).bounds().ToString());
 }
 
-TEST_P(DisplayManagerTest, LayoutMorethanThreeDisplaysTest) {
+TEST_F(DisplayManagerTest, LayoutMorethanThreeDisplaysTest) {
   int64_t primary_id = display::Screen::GetScreen()->GetPrimaryDisplay().id();
   display::DisplayIdList list = display::test::CreateDisplayIdListN(
       3, primary_id, primary_id + 1, primary_id + 2);
   {
     // Layout: [2]
     //         [1][P]
     display::DisplayLayoutBuilder builder(primary_id);
     builder.AddDisplayPlacement(list[1], primary_id,
                                 display::DisplayPlacement::LEFT, 10);
     builder.AddDisplayPlacement(list[2], list[1],
@@ skipping 97 matching lines @@
     EXPECT_EQ("-290,480 300x200",
               display_manager()->GetDisplayAt(3).bounds().ToString());
     // 1st is the bottom of 3rd.
     EXPECT_EQ("-280,680 320x200",
               display_manager()->GetDisplayAt(1).bounds().ToString());
   }
 }
 
 // Makes sure that layouts with overlapped displays are detected and fixed when
 // applied.
-TEST_P(DisplayManagerTest, NoOverlappedDisplays) {
+TEST_F(DisplayManagerTest, NoOverlappedDisplays) {
   int64_t primary_id = display::Screen::GetScreen()->GetPrimaryDisplay().id();
   {
     // Layout with multiple overlaps and special cases:
     //
     //            +-----+
     //       +----+-+6  |
     //       |  5 | |   |
     //  +----+----+ |   |
     //  | 7  |    | |   |
     //  +----+----+-+---+---+-+---------+
@@ skipping 266 matching lines @@
     EXPECT_EQ(3U, display_manager()->GetNumDisplays());
     EXPECT_EQ(gfx::Rect(0, 0, 696, 500),
               display_manager()->GetDisplayAt(0).bounds());
     EXPECT_EQ(gfx::Rect(0, -500, 696, 500),
               display_manager()->GetDisplayAt(1).bounds());
     EXPECT_EQ(gfx::Rect(-696, -500, 696, 500),
               display_manager()->GetDisplayAt(2).bounds());
   }
 }
 
-TEST_P(DisplayManagerTest, NoOverlappedDisplaysNotFitBetweenTwo) {
+TEST_F(DisplayManagerTest, NoOverlappedDisplaysNotFitBetweenTwo) {
   //    +------+--+----+--+------+
   //    |  1   |  |  2 |  |  3   |
   //    |      |  |    |  |      |
   //    |      |  |    |  |      |
   //    |      |  |    |  |      |
   //    +-+----+--+----+--+---+--+
   //      |         P         |
   //      |                   |
   //      |                   |
   //      |                   |
@@ skipping 33 matching lines @@
   EXPECT_EQ(gfx::Rect(0, 0, 1200, 500),
             display_manager()->GetDisplayAt(0).bounds());
   EXPECT_EQ(gfx::Rect(-110, -500, 600, 500),
             display_manager()->GetDisplayAt(1).bounds());
   EXPECT_EQ(gfx::Rect(490, -500, 600, 500),
             display_manager()->GetDisplayAt(2).bounds());
   EXPECT_EQ(gfx::Rect(1090, -500, 600, 500),
             display_manager()->GetDisplayAt(3).bounds());
 }
 
-TEST_P(DisplayManagerTest, NoOverlappedDisplaysAfterResolutionChange) {
+TEST_F(DisplayManagerTest, NoOverlappedDisplaysAfterResolutionChange) {
   // Starting with a good layout with no overlaps, test that if the resolution
   // of one of the displays is changed, it won't result in any overlaps.
   //
   //         +-------------------+
   //         |         4         |
   //         |                   |
   //         |                   |
   //         |                   |
   //    +----+----+---------+----+----+
   //    |    1    |    2    |    3    |
@@ skipping 68 matching lines @@
   EXPECT_EQ(gfx::Rect(-250, -500, 500, 500),
             display_manager()->GetDisplayAt(1).bounds());
   EXPECT_EQ(gfx::Rect(250, -600, 600, 600),
             display_manager()->GetDisplayAt(2).bounds());
   EXPECT_EQ(gfx::Rect(850, -500, 500, 500),
             display_manager()->GetDisplayAt(3).bounds());
   EXPECT_EQ(gfx::Rect(0, -1100, 1000, 500),
             display_manager()->GetDisplayAt(4).bounds());
 }
 
-TEST_P(DisplayManagerTest, NoOverlappedDisplaysWithDetachedDisplays) {
+TEST_F(DisplayManagerTest, NoOverlappedDisplaysWithDetachedDisplays) {
   // Detached displays that intersect other non-detached displays.
   //
   //    +---------+---------+---------+
   //    |    1    |    2    |    3    |
   //    |         |         |         |
   //    |         |         |         |
   //    |         |         |         |
   //    +----+----+-----+---+----+----+
   //         |  4, 5    | P      |
   //         | detached |        |
@@ skipping 66 matching lines @@
   expected_layout_builder.AddDisplayPlacement(
       list[4], primary_id, display::DisplayPlacement::BOTTOM, 0);
   expected_layout_builder.AddDisplayPlacement(
       list[5], list[4], display::DisplayPlacement::BOTTOM, 0);
 
   const display::DisplayLayout& layout =
       display_manager()->GetCurrentResolvedDisplayLayout();
   EXPECT_TRUE(layout.HasSamePlacementList(*(expected_layout_builder.Build())));
 }
 
-TEST_P(DisplayManagerTest, NoMirrorInThreeDisplays) {
+TEST_F(DisplayManagerTest, NoMirrorInThreeDisplays) {
   UpdateDisplay("640x480,320x200,400x300");
   ash::Shell::GetInstance()->display_configuration_controller()->SetMirrorMode(
       true, true);
   EXPECT_FALSE(display_manager()->IsInMirrorMode());
   EXPECT_EQ(3u, display_manager()->GetNumDisplays());
   EXPECT_EQ(l10n_util::GetStringUTF16(IDS_ASH_DISPLAY_MIRRORING_NOT_SUPPORTED),
             GetDisplayErrorNotificationMessageForTest());
 }
 
-TEST_P(DisplayManagerTest, OverscanInsetsTest) {
+TEST_F(DisplayManagerTest, OverscanInsetsTest) {
   UpdateDisplay("0+0-500x500,0+501-400x400");
   reset();
   ASSERT_EQ(2u, display_manager()->GetNumDisplays());
   const display::ManagedDisplayInfo display_info1 = GetDisplayInfoAt(0);
   const display::ManagedDisplayInfo display_info2 = GetDisplayInfoAt(1);
 
   display_manager()->SetOverscanInsets(display_info2.id(),
                                        gfx::Insets(13, 12, 11, 10));
 
   std::vector<display::Display> changed_displays = changed();
@@ skipping 101 matching lines @@
   display_manager()->SetOverscanInsets(primary_id, gfx::Insets(0, 0, 0, 0));
   EXPECT_TRUE(changed_metrics() &
               display::DisplayObserver::DISPLAY_METRIC_BOUNDS);
   EXPECT_TRUE(changed_metrics() &
               display::DisplayObserver::DISPLAY_METRIC_WORK_AREA);
   EXPECT_EQ(
       "0,0 500x500",
       display::Screen::GetScreen()->GetPrimaryDisplay().bounds().ToString());
 }
 
-TEST_P(DisplayManagerTest, ZeroOverscanInsets) {
+TEST_F(DisplayManagerTest, ZeroOverscanInsets) {
   // Make sure the display change events is emitted for overscan inset changes.
   UpdateDisplay("0+0-500x500,0+501-400x400");
   ASSERT_EQ(2u, display_manager()->GetNumDisplays());
   int64_t display2_id = display_manager()->GetDisplayAt(1).id();
 
   reset();
   display_manager()->SetOverscanInsets(display2_id, gfx::Insets(0, 0, 0, 0));
   EXPECT_EQ(0u, changed().size());
 
   reset();
   display_manager()->SetOverscanInsets(display2_id, gfx::Insets(1, 0, 0, 0));
   EXPECT_EQ(1u, changed().size());
   EXPECT_EQ(display2_id, changed()[0].id());
 
   reset();
   display_manager()->SetOverscanInsets(display2_id, gfx::Insets(0, 0, 0, 0));
   EXPECT_EQ(1u, changed().size());
   EXPECT_EQ(display2_id, changed()[0].id());
 }
 
-TEST_P(DisplayManagerTest, TouchCalibrationTest) {
+TEST_F(DisplayManagerTest, TouchCalibrationTest) {
   UpdateDisplay("0+0-500x500,0+501-1024x600");
   reset();
 
   ASSERT_EQ(2u, display_manager()->GetNumDisplays());
   const display::ManagedDisplayInfo display_info1 = GetDisplayInfoAt(0);
   const display::ManagedDisplayInfo display_info2 = GetDisplayInfoAt(1);
 
   EXPECT_FALSE(display_info2.has_touch_calibration_data());
 
   display::TouchCalibrationData::CalibrationPointPairQuad point_pair_quad = {
@@ skipping 49 matching lines @@
   std::vector<display::ManagedDisplayInfo> display_info_list;
   display_info_list.push_back(display_info1);
   display_info_list.push_back(display_info2);
   display_manager()->OnNativeDisplaysChanged(display_info_list);
   updated_display_info2 = GetDisplayInfoAt(1);
 
   EXPECT_FALSE(updated_display_info2.has_touch_calibration_data());
   EXPECT_EQ(touch_data, updated_display_info2.GetTouchCalibrationData());
 }
 
-TEST_P(DisplayManagerTest, TestDeviceScaleOnlyChange) {
+TEST_F(DisplayManagerTest, TestDeviceScaleOnlyChange) {
   UpdateDisplay("1000x600");
   aura::WindowTreeHost* host = Shell::GetPrimaryRootWindow()->GetHost();
   EXPECT_EQ(1, host->compositor()->device_scale_factor());
   EXPECT_EQ("1000x600",
             Shell::GetPrimaryRootWindow()->bounds().size().ToString());
   EXPECT_EQ("1 0 0", GetCountSummary());
 
   UpdateDisplay("1000x600*2");
   EXPECT_EQ(2, host->compositor()->device_scale_factor());
   EXPECT_EQ("2 0 0", GetCountSummary());
   EXPECT_EQ("500x300",
             Shell::GetPrimaryRootWindow()->bounds().size().ToString());
 }
 
 display::ManagedDisplayInfo CreateDisplayInfo(int64_t id,
                                               const gfx::Rect& bounds) {
   display::ManagedDisplayInfo info(id, ToDisplayName(id), false);
   info.SetBounds(bounds);
   return info;
 }
 
-TEST_P(DisplayManagerTest, TestNativeDisplaysChanged) {
+TEST_F(DisplayManagerTest, TestNativeDisplaysChanged) {
   const int64_t internal_display_id =
       display::test::DisplayManagerTestApi(display_manager())
           .SetFirstDisplayAsInternalDisplay();
   const int external_id = 10;
   const int mirror_id = 11;
   const int64_t invalid_id = display::kInvalidDisplayId;
   const display::ManagedDisplayInfo internal_display_info =
       CreateDisplayInfo(internal_display_id, gfx::Rect(0, 0, 500, 500));
   const display::ManagedDisplayInfo external_display_info =
       CreateDisplayInfo(external_id, gfx::Rect(1, 1, 100, 100));
@@ skipping 144 matching lines @@
   EXPECT_EQ(1U, display_manager()->GetNumDisplays());
   EXPECT_EQ(
       "0,0 500x500",
       GetDisplayInfoForId(internal_display_id).bounds_in_native().ToString());
   EXPECT_EQ(1U, display_manager()->num_connected_displays());
   EXPECT_FALSE(display_manager()->IsInMirrorMode());
 }
 
 // Make sure crash does not happen if add and remove happens at the same time.
 // See: crbug.com/414394
-TEST_P(DisplayManagerTest, DisplayAddRemoveAtTheSameTime) {
+TEST_F(DisplayManagerTest, DisplayAddRemoveAtTheSameTime) {
   UpdateDisplay("100+0-500x500,0+501-400x400");
 
   const int64_t primary_id = WindowTreeHostManager::GetPrimaryDisplayId();
   const int64_t secondary_id = display_manager()->GetSecondaryDisplay().id();
 
   display::ManagedDisplayInfo primary_info =
       display_manager()->GetDisplayInfo(primary_id);
   display::ManagedDisplayInfo secondary_info =
       display_manager()->GetDisplayInfo(secondary_id);
 
   // An id which is different from primary and secondary.
   const int64_t third_id = secondary_id + 1;
 
   display::ManagedDisplayInfo third_info =
       CreateDisplayInfo(third_id, gfx::Rect(0, 0, 600, 600));
 
   std::vector<display::ManagedDisplayInfo> display_info_list;
   display_info_list.push_back(third_info);
   display_info_list.push_back(secondary_info);
   display_manager()->OnNativeDisplaysChanged(display_info_list);
 
   // Secondary seconary_id becomes the primary as it has smaller output index.
   EXPECT_EQ(secondary_id, WindowTreeHostManager::GetPrimaryDisplayId());
   EXPECT_EQ(third_id, display_manager()->GetSecondaryDisplay().id());
   EXPECT_EQ("600x600", GetDisplayForId(third_id).size().ToString());
 }
 
-TEST_P(DisplayManagerTest, TestNativeDisplaysChangedNoInternal) {
+TEST_F(DisplayManagerTest, TestNativeDisplaysChangedNoInternal) {
   EXPECT_EQ(1U, display_manager()->GetNumDisplays());
 
   // Don't change the display info if all displays are disconnected.
   std::vector<display::ManagedDisplayInfo> display_info_list;
   display_manager()->OnNativeDisplaysChanged(display_info_list);
   EXPECT_EQ(1U, display_manager()->GetNumDisplays());
 
   // Connect another display which will become primary.
   const display::ManagedDisplayInfo external_display_info =
       CreateDisplayInfo(10, gfx::Rect(1, 1, 100, 100));
   display_info_list.push_back(external_display_info);
   display_manager()->OnNativeDisplaysChanged(display_info_list);
   EXPECT_EQ(1U, display_manager()->GetNumDisplays());
   EXPECT_EQ("1,1 100x100",
             GetDisplayInfoForId(10).bounds_in_native().ToString());
   EXPECT_EQ("100x100", ash::Shell::GetPrimaryRootWindow()
                            ->GetHost()
                            ->GetBoundsInPixels()
                            .size()
                            .ToString());
 }
 
-TEST_P(DisplayManagerTest, NativeDisplaysChangedAfterPrimaryChange) {
+TEST_F(DisplayManagerTest, NativeDisplaysChangedAfterPrimaryChange) {
   const int64_t internal_display_id =
       display::test::DisplayManagerTestApi(display_manager())
           .SetFirstDisplayAsInternalDisplay();
   const display::ManagedDisplayInfo native_display_info =
       CreateDisplayInfo(internal_display_id, gfx::Rect(0, 0, 500, 500));
   const display::ManagedDisplayInfo secondary_display_info =
       CreateDisplayInfo(10, gfx::Rect(1, 1, 100, 100));
 
   std::vector<display::ManagedDisplayInfo> display_info_list;
   display_info_list.push_back(native_display_info);
@@ skipping 11 matching lines @@
   EXPECT_EQ("0,0 100x100", GetDisplayForId(10).bounds().ToString());
 
   // OnNativeDisplaysChanged may change the display bounds.  Here makes sure
   // nothing changed if the exactly same displays are specified.
   display_manager()->OnNativeDisplaysChanged(display_info_list);
   EXPECT_EQ("-500,0 500x500",
             GetDisplayForId(internal_display_id).bounds().ToString());
   EXPECT_EQ("0,0 100x100", GetDisplayForId(10).bounds().ToString());
 }
 
-TEST_P(DisplayManagerTest, DontRememberBestResolution) {
+TEST_F(DisplayManagerTest, DontRememberBestResolution) {
   int display_id = 1000;
   display::ManagedDisplayInfo native_display_info =
       CreateDisplayInfo(display_id, gfx::Rect(0, 0, 1000, 500));
   display::ManagedDisplayInfo::ManagedDisplayModeList display_modes;
   display_modes.push_back(make_scoped_refptr(new display::ManagedDisplayMode(
       gfx::Size(1000, 500), 58.0f, false, true)));
   display_modes.push_back(make_scoped_refptr(new display::ManagedDisplayMode(
       gfx::Size(800, 300), 59.0f, false, false)));
   display_modes.push_back(make_scoped_refptr(new display::ManagedDisplayMode(
       gfx::Size(400, 500), 60.0f, false, false)));
@@ skipping 46 matching lines @@
   EXPECT_EQ(58.0f, mode->refresh_rate());
   EXPECT_TRUE(mode->native());
 
   expected_mode =
       new display::ManagedDisplayMode(gfx::Size(1000, 500), 0.0f, false, false);
 
   EXPECT_TRUE(expected_mode->IsEquivalent(
       display_manager()->GetActiveModeForDisplayId(display_id)));
 }
 
-TEST_P(DisplayManagerTest, ResolutionFallback) {
+TEST_F(DisplayManagerTest, ResolutionFallback) {
   int display_id = 1000;
   display::ManagedDisplayInfo native_display_info =
       CreateDisplayInfo(display_id, gfx::Rect(0, 0, 1000, 500));
   display::ManagedDisplayInfo::ManagedDisplayModeList display_modes;
   display_modes.push_back(make_scoped_refptr(new display::ManagedDisplayMode(
       gfx::Size(1000, 500), 58.0f, false, true)));
   display_modes.push_back(make_scoped_refptr(new display::ManagedDisplayMode(
       gfx::Size(800, 300), 59.0f, false, false)));
   display_modes.push_back(make_scoped_refptr(new display::ManagedDisplayMode(
       gfx::Size(400, 500), 60.0f, false, false)));
@@ skipping 36 matching lines @@
 
     scoped_refptr<display::ManagedDisplayMode> mode =
         display_manager()->GetSelectedModeForDisplayId(display_id);
     EXPECT_TRUE(!!mode);
     EXPECT_EQ("1000x500", mode->size().ToString());
     EXPECT_EQ(58.0f, mode->refresh_rate());
     EXPECT_TRUE(mode->native());
   }
 }
 
-TEST_P(DisplayManagerTest, Rotate) {
+TEST_F(DisplayManagerTest, Rotate) {
   UpdateDisplay("100x200/r,300x400/l");
   EXPECT_EQ("1,1 100x200", GetDisplayInfoAt(0).bounds_in_native().ToString());
   EXPECT_EQ("200x100", GetDisplayInfoAt(0).size_in_pixel().ToString());
 
   EXPECT_EQ("1,201 300x400", GetDisplayInfoAt(1).bounds_in_native().ToString());
   EXPECT_EQ("400x300", GetDisplayInfoAt(1).size_in_pixel().ToString());
   reset();
   UpdateDisplay("100x200/b,300x400");
   EXPECT_EQ("2 0 0", GetCountSummary());
   reset();
@@ skipping 64 matching lines @@
       internal_display_id, display::Display::ROTATE_180,
       display::Display::ROTATION_SOURCE_ACTIVE);
   const display::ManagedDisplayInfo& post_rotation_info =
       display::test::DisplayManagerTestApi(display_manager())
           .GetInternalManagedDisplayInfo(internal_display_id);
   EXPECT_NE(info.GetActiveRotation(), post_rotation_info.GetActiveRotation());
   EXPECT_EQ(display::Display::ROTATE_180,
             post_rotation_info.GetActiveRotation());
 }
 
-TEST_P(DisplayManagerTest, UIScale) {
+TEST_F(DisplayManagerTest, UIScale) {
   display::test::ScopedDisable125DSFForUIScaling disable;
 
   UpdateDisplay("1280x800");
   int64_t display_id = display::Screen::GetScreen()->GetPrimaryDisplay().id();
   display::test::DisplayManagerTestApi(display_manager())
       .SetDisplayUIScale(display_id, 1.125f);
   EXPECT_EQ(1.0, GetDisplayInfoAt(0).configured_ui_scale());
   display::test::DisplayManagerTestApi(display_manager())
       .SetDisplayUIScale(display_id, 0.8f);
   EXPECT_EQ(1.0f, GetDisplayInfoAt(0).configured_ui_scale());
@@ skipping 105 matching lines @@
   UpdateDisplay("1280x850*1.25");
   display::test::DisplayManagerTestApi(display_manager())
       .SetDisplayUIScale(display_id, 1.25f);
   EXPECT_EQ(1.25f, GetDisplayInfoAt(0).configured_ui_scale());
   EXPECT_EQ(1.0f, GetDisplayInfoAt(0).GetEffectiveUIScale());
   display = display::Screen::GetScreen()->GetPrimaryDisplay();
   EXPECT_EQ(1.0f, display.device_scale_factor());
   EXPECT_EQ("1280x850", display.bounds().size().ToString());
 }
 
-TEST_P(DisplayManagerTest, UIScaleWithDisplayMode) {
+TEST_F(DisplayManagerTest, UIScaleWithDisplayMode) {
   int display_id = 1000;
 
   // Setup the display modes with UI-scale.
   display::ManagedDisplayInfo native_display_info =
       CreateDisplayInfo(display_id, gfx::Rect(0, 0, 1280, 800));
   const scoped_refptr<display::ManagedDisplayMode>& base_mode(
       new display::ManagedDisplayMode(gfx::Size(1280, 800), 60.0f, false,
                                       false));
   display::ManagedDisplayInfo::ManagedDisplayModeList mode_list =
       CreateInternalManagedDisplayModeList(base_mode);
@@ skipping 71 matching lines @@
       expected_mode->size(), expected_mode->refresh_rate(),
       expected_mode->is_interlaced(), expected_mode->native(),
       0.5f /* ui_scale */, expected_mode->device_scale_factor());
 
   EXPECT_TRUE(expected_mode->IsEquivalent(
       display_manager()->GetActiveModeForDisplayId(display_id)));
 }
 
 // Tests that ResetInternalDisplayZoom() resets to the default 0.8f UI scale
 // defined for the 1.25x displays.
-TEST_P(DisplayManagerTest, ResetInternalDisplayZoomFor1_25x) {
+TEST_F(DisplayManagerTest, ResetInternalDisplayZoomFor1_25x) {
   // Setup the display modes with UI-scale.
   const scoped_refptr<display::ManagedDisplayMode> base_mode(
       new display::ManagedDisplayMode(gfx::Size(1920, 1080), 60.0f,
                                       false /* is_interlaced */,
                                       true /* native */, 1.0f /* ui_scale */,
                                       1.25f /* device_scale_factor */));
   display::ManagedDisplayInfo::ManagedDisplayModeList mode_list =
       CreateInternalManagedDisplayModeList(base_mode);
 
   const int display_id = 1000;
@@ skipping 22 matching lines @@
 
   // Reset the internal display zoom and expect the UI scale to go to the
   // default 0.8f.
   display_manager()->ResetInternalDisplayZoom();
   EXPECT_EQ(1.25f, GetDisplayInfoAt(0).GetEffectiveDeviceScaleFactor());
   EXPECT_EQ(1.0f, GetDisplayInfoAt(0).GetEffectiveUIScale());
   EXPECT_EQ(0.8f, GetDisplayInfoAt(0).configured_ui_scale());
   EXPECT_EQ("1536x864", GetDisplayForId(display_id).size().ToString());
 }
 
-TEST_P(DisplayManagerTest, Use125DSFForUIScaling) {
+TEST_F(DisplayManagerTest, Use125DSFForUIScaling) {
   int64_t display_id = display::Screen::GetScreen()->GetPrimaryDisplay().id();
 
   display::test::ScopedSetInternalDisplayId set_internal(display_manager(),
                                                          display_id);
   UpdateDisplay("1920x1080*1.25");
   EXPECT_EQ(1.0f, GetDisplayInfoAt(0).GetEffectiveDeviceScaleFactor());
   EXPECT_EQ(1.0f, GetDisplayInfoAt(0).GetEffectiveUIScale());
 
   display::test::DisplayManagerTestApi(display_manager())
       .SetDisplayUIScale(display_id, 0.8f);
   EXPECT_EQ(1.25f, GetDisplayInfoAt(0).GetEffectiveDeviceScaleFactor());
   EXPECT_EQ(1.0f, GetDisplayInfoAt(0).GetEffectiveUIScale());
   EXPECT_EQ("1536x864", GetDisplayForId(display_id).size().ToString());
 
   display::test::DisplayManagerTestApi(display_manager())
       .SetDisplayUIScale(display_id, 0.5f);
   EXPECT_EQ(1.0f, GetDisplayInfoAt(0).GetEffectiveDeviceScaleFactor());
   EXPECT_EQ(0.5f, GetDisplayInfoAt(0).GetEffectiveUIScale());
   EXPECT_EQ("960x540", GetDisplayForId(display_id).size().ToString());
 
   display::test::DisplayManagerTestApi(display_manager())
       .SetDisplayUIScale(display_id, 1.25f);
   EXPECT_EQ(1.0f, GetDisplayInfoAt(0).GetEffectiveDeviceScaleFactor());
   EXPECT_EQ(1.25f, GetDisplayInfoAt(0).GetEffectiveUIScale());
   EXPECT_EQ("2400x1350", GetDisplayForId(display_id).size().ToString());
 }
 
-TEST_P(DisplayManagerTest, FHD125DefaultsTo08UIScaling) {
+TEST_F(DisplayManagerTest, FHD125DefaultsTo08UIScaling) {
   int64_t display_id = display::Screen::GetScreen()->GetPrimaryDisplay().id();
 
   display_id++;
   display::test::ScopedSetInternalDisplayId set_internal(display_manager(),
                                                          display_id);
 
   // Setup the display modes with UI-scale.
   display::ManagedDisplayInfo native_display_info =
       CreateDisplayInfo(display_id, gfx::Rect(0, 0, 1920, 1080));
   native_display_info.set_device_scale_factor(1.25);
 
   const scoped_refptr<display::ManagedDisplayMode>& base_mode(
       new display::ManagedDisplayMode(gfx::Size(1920, 1080), 60.0f, false,
                                       false));
   display::ManagedDisplayInfo::ManagedDisplayModeList mode_list =
       CreateInternalManagedDisplayModeList(base_mode);
   native_display_info.SetManagedDisplayModes(mode_list);
 
   std::vector<display::ManagedDisplayInfo> display_info_list;
   display_info_list.push_back(native_display_info);
 
   display_manager()->OnNativeDisplaysChanged(display_info_list);
 
   EXPECT_EQ(1.25f, GetDisplayInfoAt(0).GetEffectiveDeviceScaleFactor());
   EXPECT_EQ(1.0f, GetDisplayInfoAt(0).GetEffectiveUIScale());
 }
 
 // Don't default to 1.25 DSF if the user already has a prefrence stored for
 // the internal display.
-TEST_P(DisplayManagerTest, FHD125DefaultsTo08UIScalingNoOverride) {
+TEST_F(DisplayManagerTest, FHD125DefaultsTo08UIScalingNoOverride) {
   int64_t display_id = display::Screen::GetScreen()->GetPrimaryDisplay().id();
 
   display_id++;
   display::test::ScopedSetInternalDisplayId set_internal(display_manager(),
                                                          display_id);
   const gfx::Insets dummy_overscan_insets;
   display_manager()->RegisterDisplayProperty(
       display_id, display::Display::ROTATE_0, 1.0f, &dummy_overscan_insets,
       gfx::Size(), 1.0f, display::ColorCalibrationProfile(), nullptr);
 
@@ skipping 11 matching lines @@
 
   std::vector<display::ManagedDisplayInfo> display_info_list;
   display_info_list.push_back(native_display_info);
 
   display_manager()->OnNativeDisplaysChanged(display_info_list);
 
   EXPECT_EQ(1.0f, GetDisplayInfoAt(0).GetEffectiveDeviceScaleFactor());
   EXPECT_EQ(1.0f, GetDisplayInfoAt(0).GetEffectiveUIScale());
 }
 
-TEST_P(DisplayManagerTest, ResolutionChangeInUnifiedMode) {
+TEST_F(DisplayManagerTest, ResolutionChangeInUnifiedMode) {
   // Don't check root window destruction in unified mode.
   Shell::GetPrimaryRootWindow()->RemoveObserver(this);
 
   display_manager()->SetUnifiedDesktopEnabled(true);
 
   UpdateDisplay("200x200, 400x400");
 
   int64_t unified_id = display::Screen::GetScreen()->GetPrimaryDisplay().id();
   display::ManagedDisplayInfo info =
       display_manager()->GetDisplayInfo(unified_id);
@@ skipping 24 matching lines @@
   UpdateDisplay("600x600, 200x200");
   EXPECT_EQ(
       "1200x600",
       display::Screen::GetScreen()->GetPrimaryDisplay().size().ToString());
   active_mode = display_manager()->GetActiveModeForDisplayId(unified_id);
   EXPECT_EQ(1.0f, active_mode->ui_scale());
   EXPECT_TRUE(active_mode->native());
   EXPECT_EQ("1200x600", active_mode->size().ToString());
 }
 
-TEST_P(DisplayManagerTest, UpdateMouseCursorAfterRotateZoom) {
+TEST_F(DisplayManagerTest, UpdateMouseCursorAfterRotateZoom) {
   // Make sure just rotating will not change native location.
   UpdateDisplay("300x200,200x150");
   aura::Window::Windows root_windows = Shell::GetAllRootWindows();
   aura::Env* env = aura::Env::GetInstance();
 
   ui::test::EventGenerator generator1(root_windows[0]);
   ui::test::EventGenerator generator2(root_windows[1]);
 
   // Test on 1st display.
   generator1.MoveMouseToInHost(150, 50);
@@ skipping 60 matching lines @@
     return changed;
   }
 
  private:
   test::MirrorWindowTestApi test_api;
   bool changed_;
 
   DISALLOW_COPY_AND_ASSIGN(TestDisplayObserver);
 };
 
-TEST_P(DisplayManagerTest, SoftwareMirroring) {
+TEST_F(DisplayManagerTest, SoftwareMirroring) {
   UpdateDisplay("300x400,400x500");
 
   test::MirrorWindowTestApi test_api;
   EXPECT_EQ(nullptr, test_api.GetHost());
 
   TestDisplayObserver display_observer;
   display::Screen::GetScreen()->AddObserver(&display_observer);
 
   display_manager()->SetMultiDisplayMode(display::DisplayManager::MIRRORING);
   display_manager()->UpdateDisplays();
@@ skipping 42 matching lines @@
 
   // Overscan insets are ignored.
   UpdateDisplay("400x600/o,600x800/o");
   EXPECT_FALSE(display_observer.changed_and_reset());
   EXPECT_EQ("400x600",
             test_api.GetHost()->window()->bounds().size().ToString());
 
   display::Screen::GetScreen()->RemoveObserver(&display_observer);
 }
 
-TEST_P(DisplayManagerTest, RotateInSoftwareMirroring) {
+TEST_F(DisplayManagerTest, RotateInSoftwareMirroring) {
   UpdateDisplay("600x400,500x300");
   display_manager()->SetMirrorMode(true);
 
   EXPECT_EQ(1U, display_manager()->GetNumDisplays());
   int64_t primary_id = display::Screen::GetScreen()->GetPrimaryDisplay().id();
   display_manager()->SetDisplayRotation(
       primary_id, display::Display::ROTATE_180,
       display::Display::ROTATION_SOURCE_ACTIVE);
   display_manager()->SetMirrorMode(false);
 }
 
-TEST_P(DisplayManagerTest, SingleDisplayToSoftwareMirroring) {
+TEST_F(DisplayManagerTest, SingleDisplayToSoftwareMirroring) {
   UpdateDisplay("600x400");
 
   display_manager()->SetMultiDisplayMode(display::DisplayManager::MIRRORING);
   UpdateDisplay("600x400,600x400");
 
   EXPECT_TRUE(display_manager()->IsInMirrorMode());
   EXPECT_EQ(1U, display_manager()->GetNumDisplays());
   WindowTreeHostManager* window_tree_host_manager =
       ash::Shell::GetInstance()->window_tree_host_manager();
   EXPECT_TRUE(
       window_tree_host_manager->mirror_window_controller()->GetWindow());
 
   UpdateDisplay("600x400");
   EXPECT_FALSE(display_manager()->IsInMirrorMode());
   EXPECT_EQ(1U, display_manager()->GetNumDisplays());
   EXPECT_FALSE(
       window_tree_host_manager->mirror_window_controller()->GetWindow());
 }
 
 // Make sure this does not cause any crashes. See http://crbug.com/412910
-TEST_P(DisplayManagerTest, SoftwareMirroringWithCompositingCursor) {
+TEST_F(DisplayManagerTest, SoftwareMirroringWithCompositingCursor) {
   UpdateDisplay("300x400,400x500");
 
   test::MirrorWindowTestApi test_api;
   EXPECT_EQ(nullptr, test_api.GetHost());
 
   display::ManagedDisplayInfo secondary_info =
       display_manager()->GetDisplayInfo(
           display_manager()->GetSecondaryDisplay().id());
 
   display_manager()->SetSoftwareMirroring(true);
@@ skipping 11 matching lines @@
   std::vector<display::ManagedDisplayInfo> new_info_list;
   new_info_list.push_back(secondary_info);
   display_manager()->OnNativeDisplaysChanged(new_info_list);
 
   root_windows = Shell::GetAllRootWindows();
   EXPECT_TRUE(root_windows[0]->Contains(test_api.GetCursorWindow()));
 
   Shell::GetInstance()->SetCursorCompositingEnabled(false);
 }
 
-TEST_P(DisplayManagerTest, MirroredLayout) {
+TEST_F(DisplayManagerTest, MirroredLayout) {
   UpdateDisplay("500x500,400x400");
   EXPECT_FALSE(display_manager()->GetCurrentDisplayLayout().mirrored);
   EXPECT_EQ(2, display::Screen::GetScreen()->GetNumDisplays());
   EXPECT_EQ(2U, display_manager()->num_connected_displays());
 
   UpdateDisplay("1+0-500x500,1+0-500x500");
   EXPECT_TRUE(display_manager()->GetCurrentDisplayLayout().mirrored);
   EXPECT_EQ(1, display::Screen::GetScreen()->GetNumDisplays());
   EXPECT_EQ(2U, display_manager()->num_connected_displays());
 
   UpdateDisplay("500x500,500x500");
   EXPECT_FALSE(display_manager()->GetCurrentDisplayLayout().mirrored);
   EXPECT_EQ(2, display::Screen::GetScreen()->GetNumDisplays());
   EXPECT_EQ(2U, display_manager()->num_connected_displays());
 }
 
-TEST_P(DisplayManagerTest, InvertLayout) {
+TEST_F(DisplayManagerTest, InvertLayout) {
   EXPECT_EQ("left, 0",
             display::DisplayPlacement(display::DisplayPlacement::RIGHT, 0)
                 .Swap()
                 .ToString());
   EXPECT_EQ("left, -100",
             display::DisplayPlacement(display::DisplayPlacement::RIGHT, 100)
                 .Swap()
                 .ToString());
   EXPECT_EQ("left, 50",
             display::DisplayPlacement(display::DisplayPlacement::RIGHT, -50)
@@ skipping 33 matching lines @@
   EXPECT_EQ("top, -70",
             display::DisplayPlacement(display::DisplayPlacement::BOTTOM, 70)
                 .Swap()
                 .ToString());
   EXPECT_EQ("top, 80",
             display::DisplayPlacement(display::DisplayPlacement::BOTTOM, -80)
                 .Swap()
                 .ToString());
 }
 
-TEST_P(DisplayManagerTest, NotifyPrimaryChange) {
+TEST_F(DisplayManagerTest, NotifyPrimaryChange) {
   UpdateDisplay("500x500,500x500");
   SwapPrimaryDisplay();
   reset();
   UpdateDisplay("500x500");
   EXPECT_FALSE(changed_metrics() &
                display::DisplayObserver::DISPLAY_METRIC_BOUNDS);
   EXPECT_FALSE(changed_metrics() &
                display::DisplayObserver::DISPLAY_METRIC_WORK_AREA);
   EXPECT_TRUE(changed_metrics() &
               display::DisplayObserver::DISPLAY_METRIC_PRIMARY);
 
   UpdateDisplay("500x500,500x500");
   SwapPrimaryDisplay();
   UpdateDisplay("500x400");
   EXPECT_TRUE(changed_metrics() &
               display::DisplayObserver::DISPLAY_METRIC_BOUNDS);
   EXPECT_TRUE(changed_metrics() &
               display::DisplayObserver::DISPLAY_METRIC_WORK_AREA);
   EXPECT_TRUE(changed_metrics() &
               display::DisplayObserver::DISPLAY_METRIC_PRIMARY);
 }
 
-TEST_P(DisplayManagerTest, NotifyPrimaryChangeUndock) {
+TEST_F(DisplayManagerTest, NotifyPrimaryChangeUndock) {
   // Assume the default display is an external display, and
   // emulates undocking by switching to another display.
   display::ManagedDisplayInfo another_display_info =
       CreateDisplayInfo(1, gfx::Rect(0, 0, 1280, 800));
   std::vector<display::ManagedDisplayInfo> info_list;
   info_list.push_back(another_display_info);
   reset();
   display_manager()->OnNativeDisplaysChanged(info_list);
   EXPECT_TRUE(changed_metrics() &
               display::DisplayObserver::DISPLAY_METRIC_BOUNDS);
   EXPECT_TRUE(changed_metrics() &
               display::DisplayObserver::DISPLAY_METRIC_WORK_AREA);
   EXPECT_TRUE(changed_metrics() &
               display::DisplayObserver::DISPLAY_METRIC_PRIMARY);
 }
 
-TEST_P(DisplayManagerTest, UpdateDisplayWithHostOrigin) {
+TEST_F(DisplayManagerTest, UpdateDisplayWithHostOrigin) {
   UpdateDisplay("100x200,300x400");
   ASSERT_EQ(2, display::Screen::GetScreen()->GetNumDisplays());
   aura::Window::Windows root_windows =
       Shell::GetInstance()->GetAllRootWindows();
   ASSERT_EQ(2U, root_windows.size());
   aura::WindowTreeHost* host0 = root_windows[0]->GetHost();
   aura::WindowTreeHost* host1 = root_windows[1]->GetHost();
 
   EXPECT_EQ("1,1", host0->GetBoundsInPixels().origin().ToString());
   EXPECT_EQ("100x200", host0->GetBoundsInPixels().size().ToString());
@@ skipping 16 matching lines @@
   EXPECT_EQ("300x400", host1->GetBoundsInPixels().size().ToString());
 
   UpdateDisplay("100+200-100x200,300+500-200x300");
   ASSERT_EQ(2, display::Screen::GetScreen()->GetNumDisplays());
   EXPECT_EQ("100,200", host0->GetBoundsInPixels().origin().ToString());
   EXPECT_EQ("100x200", host0->GetBoundsInPixels().size().ToString());
   EXPECT_EQ("300,500", host1->GetBoundsInPixels().origin().ToString());
   EXPECT_EQ("200x300", host1->GetBoundsInPixels().size().ToString());
 }
 
-TEST_P(DisplayManagerTest, UnifiedDesktopBasic) {
+TEST_F(DisplayManagerTest, UnifiedDesktopBasic) {
   // Don't check root window destruction in unified mode.
   Shell::GetPrimaryRootWindow()->RemoveObserver(this);
 
   UpdateDisplay("400x500,300x200");
 
   // Enable after extended mode.
   display_manager()->SetUnifiedDesktopEnabled(true);
 
   // Defaults to the unified desktop.
   display::Screen* screen = display::Screen::GetScreen();
@@ skipping 38 matching lines @@
   EXPECT_EQ(gfx::Size(500, 300), screen->GetPrimaryDisplay().size());
   EXPECT_EQ(gfx::Size(400, 500),
             display_manager()->GetSecondaryDisplay().size());
   EXPECT_EQ(
       gfx::Size(500, 300),
       display_manager()
           ->GetDisplayForId(display_manager()->GetSecondaryDisplay().id() + 1)
           .size());
 }
 
-TEST_P(DisplayManagerTest, UnifiedDesktopWithHardwareMirroring) {
+TEST_F(DisplayManagerTest, UnifiedDesktopWithHardwareMirroring) {
   // Don't check root window destruction in unified mode.
   Shell::GetPrimaryRootWindow()->RemoveObserver(this);
 
   // Enter to hardware mirroring.
   display::ManagedDisplayInfo d1(1, "", false);
   d1.SetBounds(gfx::Rect(0, 0, 500, 500));
   display::ManagedDisplayInfo d2(2, "", false);
   d2.SetBounds(gfx::Rect(0, 0, 500, 500));
   std::vector<display::ManagedDisplayInfo> display_info_list;
   display_info_list.push_back(d1);
@@ skipping 17 matching lines @@
   // Exit from hardware mirroring.
   d2.SetBounds(gfx::Rect(0, 500, 500, 500));
   display_info_list.clear();
   display_info_list.push_back(d1);
   display_info_list.push_back(d2);
   display_manager()->OnNativeDisplaysChanged(display_info_list);
   EXPECT_FALSE(display_manager()->IsInMirrorMode());
   EXPECT_TRUE(display_manager()->IsInUnifiedMode());
 }
 
-TEST_P(DisplayManagerTest, UnifiedDesktopEnabledWithExtended) {
+TEST_F(DisplayManagerTest, UnifiedDesktopEnabledWithExtended) {
   // Don't check root window destruction in unified mode.
   Shell::GetPrimaryRootWindow()->RemoveObserver(this);
 
   UpdateDisplay("400x500,300x200");
   display::DisplayIdList list = display_manager()->GetCurrentDisplayIdList();
   display::DisplayLayoutBuilder builder(
       display_manager()->layout_store()->GetRegisteredDisplayLayout(list));
   builder.SetDefaultUnified(false);
   display_manager()->layout_store()->RegisterLayoutForDisplayIdList(
       list, builder.Build());
   display_manager()->SetUnifiedDesktopEnabled(true);
   EXPECT_FALSE(display_manager()->IsInUnifiedMode());
 }
 
-TEST_P(DisplayManagerTest, UnifiedDesktopWith2xDSF) {
+TEST_F(DisplayManagerTest, UnifiedDesktopWith2xDSF) {
   // Don't check root window destruction in unified mode.
   Shell::GetPrimaryRootWindow()->RemoveObserver(this);
 
   display_manager()->SetUnifiedDesktopEnabled(true);
   display::Screen* screen = display::Screen::GetScreen();
 
   // 2nd display is 2x.
   UpdateDisplay("400x500,1000x800*2");
   display::ManagedDisplayInfo info =
       display_manager()->GetDisplayInfo(screen->GetPrimaryDisplay().id());
@@ skipping 81 matching lines @@
   EXPECT_EQ("1300x800",
             Shell::GetPrimaryRootWindow()->bounds().size().ToString());
   accelerators::ZoomInternalDisplay(false);
   EXPECT_EQ("650x400", screen->GetPrimaryDisplay().size().ToString());
   EXPECT_EQ("650x400",
             Shell::GetPrimaryRootWindow()->bounds().size().ToString());
 }
 
 // Updating displays again in unified desktop mode should not crash.
 // crbug.com/491094.
-TEST_P(DisplayManagerTest, ConfigureUnifiedTwice) {
+TEST_F(DisplayManagerTest, ConfigureUnifiedTwice) {
   // Don't check root window destruction in unified mode.
   Shell::GetPrimaryRootWindow()->RemoveObserver(this);
 
   UpdateDisplay("300x200,400x500");
   // Mirror windows are created in a posted task.
   RunAllPendingInMessageLoop();
 
   UpdateDisplay("300x250,400x550");
   RunAllPendingInMessageLoop();
 }
 
-TEST_P(DisplayManagerTest, NoRotateUnifiedDesktop) {
+TEST_F(DisplayManagerTest, NoRotateUnifiedDesktop) {
   display_manager()->SetUnifiedDesktopEnabled(true);
 
   // Don't check root window destruction in unified mode.
   Shell::GetPrimaryRootWindow()->RemoveObserver(this);
 
   UpdateDisplay("400x500,300x200");
 
   display::Screen* screen = display::Screen::GetScreen();
   const display::Display& display = screen->GetPrimaryDisplay();
   EXPECT_EQ("1150x500", display.size().ToString());
   display_manager()->SetDisplayRotation(
       display.id(), display::Display::ROTATE_90,
       display::Display::ROTATION_SOURCE_ACTIVE);
   EXPECT_EQ("1150x500", screen->GetPrimaryDisplay().size().ToString());
   display_manager()->SetDisplayRotation(
       display.id(), display::Display::ROTATE_0,
       display::Display::ROTATION_SOURCE_ACTIVE);
   EXPECT_EQ("1150x500", screen->GetPrimaryDisplay().size().ToString());
 
   UpdateDisplay("400x500");
   EXPECT_EQ("400x500", screen->GetPrimaryDisplay().size().ToString());
 }
 
 // Makes sure the transition from unified to single won't crash
 // with docked windows.
-TEST_P(DisplayManagerTest, UnifiedWithDockWindows) {
+TEST_F(DisplayManagerTest, UnifiedWithDockWindows) {
   // Enable window docking for this test.
   base::CommandLine::ForCurrentProcess()->AppendSwitch(
       ash::switches::kAshEnableDockedWindows);
 
-  const int height_offset = GetMdMaximizedWindowHeightOffset();
   display_manager()->SetUnifiedDesktopEnabled(true);
 
   // Don't check root window destruction in unified mode.
   Shell::GetPrimaryRootWindow()->RemoveObserver(this);
 
   UpdateDisplay("400x500,300x200");
 
   std::unique_ptr<aura::Window> docked(
       CreateTestWindowInShellWithBounds(gfx::Rect(10, 10, 50, 50)));
   docked->SetProperty(aura::client::kShowStateKey, ui::SHOW_STATE_DOCKED);
   ASSERT_TRUE(wm::GetWindowState(docked.get())->IsDocked());
-  // 47 pixels reserved for launcher shelf height in non-material design, and
-  // 48 pixels reserved in material design.
-  EXPECT_EQ(gfx::Rect(0, 0, 250, 453 + height_offset).ToString(),
-            docked->bounds().ToString());
+  // 48 pixels reserved for launcher shelf height.
+  EXPECT_EQ(gfx::Rect(0, 0, 250, 452).ToString(), docked->bounds().ToString());
   UpdateDisplay("300x300");
   // Make sure the window is still docked.
   EXPECT_TRUE(wm::GetWindowState(docked.get())->IsDocked());
-  EXPECT_EQ(gfx::Rect(0, 0, 250, 253 + height_offset).ToString(),
-            docked->bounds().ToString());
+  EXPECT_EQ(gfx::Rect(0, 0, 250, 252).ToString(), docked->bounds().ToString());
 }
 
-TEST_P(DisplayManagerTest, DockMode) {
+TEST_F(DisplayManagerTest, DockMode) {
   const int64_t internal_id = 1;
   const int64_t external_id = 2;
 
   const display::ManagedDisplayInfo internal_display_info =
       CreateDisplayInfo(internal_id, gfx::Rect(0, 0, 500, 500));
   const display::ManagedDisplayInfo external_display_info =
       CreateDisplayInfo(external_id, gfx::Rect(1, 1, 100, 100));
   std::vector<display::ManagedDisplayInfo> display_info_list;
 
   // software mirroring.
@@ skipping 13 matching lines @@
   EXPECT_TRUE(display_manager()->IsActiveDisplayId(external_id));
   EXPECT_FALSE(display_manager()->IsActiveDisplayId(internal_id));
 
   EXPECT_FALSE(display_manager()->ZoomInternalDisplay(true));
   EXPECT_FALSE(display_manager()->ZoomInternalDisplay(false));
   EXPECT_FALSE(display::test::DisplayManagerTestApi(display_manager())
                    .SetDisplayUIScale(internal_id, 1.0f));
 }
 
 // Make sure that bad layout information is ignored and does not crash.
-TEST_P(DisplayManagerTest, DontRegisterBadConfig) {
+TEST_F(DisplayManagerTest, DontRegisterBadConfig) {
   display::DisplayIdList list = display::test::CreateDisplayIdList2(1, 2);
   display::DisplayLayoutBuilder builder(1);
   builder.AddDisplayPlacement(2, 1, display::DisplayPlacement::LEFT, 0);
   builder.AddDisplayPlacement(3, 1, display::DisplayPlacement::BOTTOM, 0);
 
   display_manager()->layout_store()->RegisterLayoutForDisplayIdList(
       list, builder.Build());
 }
 
 class ScreenShutdownTest : public test::AshTestBase {
@@ skipping 143 matching lines @@
       .SetDisplayUIScale(display::Screen::GetScreen()->GetPrimaryDisplay().id(),
                          0.8f);
 
   ASSERT_DOUBLE_EQ(
       1.25f,
       display::Screen::GetScreen()->GetPrimaryDisplay().device_scale_factor());
   EXPECT_TRUE(IsTextSubpixelPositioningEnabled());
   EXPECT_EQ(gfx::FontRenderParams::HINTING_NONE, GetFontHintingParams());
 }
 
-TEST_P(DisplayManagerTest, CheckInitializationOfRotationProperty) {
+TEST_F(DisplayManagerTest, CheckInitializationOfRotationProperty) {
   int64_t id = display_manager()->GetDisplayAt(0).id();
   display_manager()->RegisterDisplayProperty(
       id, display::Display::ROTATE_90, 1.0f, nullptr, gfx::Size(), 1.0f,
       display::COLOR_PROFILE_STANDARD, nullptr);
 
   const display::ManagedDisplayInfo& info =
       display_manager()->GetDisplayInfo(id);
 
   EXPECT_EQ(display::Display::ROTATE_90,
             info.GetRotation(display::Display::ROTATION_SOURCE_USER));
   EXPECT_EQ(display::Display::ROTATE_90,
             info.GetRotation(display::Display::ROTATION_SOURCE_ACTIVE));
 }
 
-TEST_P(DisplayManagerTest, RejectInvalidLayoutData) {
+TEST_F(DisplayManagerTest, RejectInvalidLayoutData) {
   display::DisplayLayoutStore* layout_store = display_manager()->layout_store();
   int64_t id1 = 10001;
   int64_t id2 = 10002;
   ASSERT_TRUE(display::CompareDisplayIds(id1, id2));
   display::DisplayLayoutBuilder good_builder(id1);
   good_builder.SetSecondaryPlacement(id2, display::DisplayPlacement::LEFT, 0);
   std::unique_ptr<display::DisplayLayout> good(good_builder.Build());
 
   display::DisplayIdList good_list =
       display::test::CreateDisplayIdList2(id1, id2);
   layout_store->RegisterLayoutForDisplayIdList(good_list, good->Copy());
 
   display::DisplayLayoutBuilder bad(id1);
   bad.SetSecondaryPlacement(id2, display::DisplayPlacement::BOTTOM, 0);
 
   display::DisplayIdList bad_list(2);
   bad_list[0] = id2;
   bad_list[1] = id1;
   layout_store->RegisterLayoutForDisplayIdList(bad_list, bad.Build());
 
   EXPECT_EQ(good->ToString(),
             layout_store->GetRegisteredDisplayLayout(good_list).ToString());
 }
 
-TEST_P(DisplayManagerTest, GuessDisplayIdFieldsInDisplayLayout) {
+TEST_F(DisplayManagerTest, GuessDisplayIdFieldsInDisplayLayout) {
   int64_t id1 = 10001;
   int64_t id2 = 10002;
 
   std::unique_ptr<display::DisplayLayout> old_layout(
       new display::DisplayLayout);
   old_layout->placement_list.emplace_back(display::DisplayPlacement::BOTTOM, 0);
   old_layout->primary_id = id1;
 
   display::DisplayLayoutStore* layout_store = display_manager()->layout_store();
   display::DisplayIdList list = display::test::CreateDisplayIdList2(id1, id2);
   layout_store->RegisterLayoutForDisplayIdList(list, std::move(old_layout));
   const display::DisplayLayout& stored =
       layout_store->GetRegisteredDisplayLayout(list);
 
   EXPECT_EQ(id1, stored.placement_list[0].parent_display_id);
   EXPECT_EQ(id2, stored.placement_list[0].display_id);
 }
 
 }  // namespace ash