chromeos: Makes AshTestBase/Helper target mash when appropriate

ash/display/window_tree_host_manager_unittest.cc

Index: ash/display/window_tree_host_manager_unittest.cc
diff --git a/ash/display/window_tree_host_manager_unittest.cc b/ash/display/window_tree_host_manager_unittest.cc
index 99d37cfea15aaf45b8f36e8639d64a0be7b2d74e..2f1a3bc9095ee845d140da31491e57a07eb8eecf 100644
--- a/ash/display/window_tree_host_manager_unittest.cc
+++ b/ash/display/window_tree_host_manager_unittest.cc
@@ -185,11 +185,6 @@ display::Display GetPrimaryDisplay() {
       Shell::GetAllRootWindows()[0]);
 }
 
-display::Display GetSecondaryDisplay() {

[msw, 2017/02/19 21:49:20]

Can you give some context for this? Do we need this on AshTestBase/Helper for
parity with AshTest? (if so, why is it needed in this CL? afaict, nothing
changes base classes here; is this just to prep for those changes?)
Does GetPrimaryDisplay also belong there? (It'd be nice to see more consistency
with those or just rely on display::Screen or similar)

[sky, 2017/02/21 17:06:50]

Exactly.
I added GetSecondaryDisplay() to AshTestBase, which manes this override is no
longer necessary.
Shell has a couple of functions for the primary window, but not the secondary.
It's common for tests to want the secondary, hence the convenience function.

-  return display::Screen::GetScreen()->GetDisplayNearestWindow(
-      Shell::GetAllRootWindows()[1]);
-}
-
 class TestHelper {
  public:
   explicit TestHelper(test::AshTestBase* delegate);