Add GetDisplayWithDisplayId to display::Screen.

ui/display/win/screen_win_unittest.cc

 // Copyright 2015 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/win/screen_win.h"
 
 #include <windows.h>
 #include <inttypes.h>
 #include <stddef.h>
 
@@ skipping 335 matching lines @@
   EXPECT_EQ(1.0, ScreenWin::GetScaleFactorForHWND(GetFakeHwnd()));
 }
 
 TEST_F(ScreenWinTestSingleDisplay1x, GetDisplays) {
   std::vector<Display> displays = GetScreen()->GetAllDisplays();
   ASSERT_EQ(1u, displays.size());
   EXPECT_EQ(gfx::Rect(0, 0, 1920, 1200), displays[0].bounds());
   EXPECT_EQ(gfx::Rect(0, 0, 1920, 1100), displays[0].work_area());
 }
 
+TEST_F(ScreenWinTestSingleDisplay1x, GetDisplayById) {
+  Screen* screen = GetScreen();
+  int64_t primary_display_id = screen->GetPrimaryDisplay().id();
+  Display display;
+  bool has_display =
+      screen->GetDisplayWithDisplayId(primary_display_id, &display);
+  EXPECT_TRUE(has_display);

[sky, 2016/11/19 15:01:40]

no need for the temporary. This comment holds for all tests where I think you
use this pattern.

+  EXPECT_EQ(screen->GetPrimaryDisplay(), display);
+}
+
 TEST_F(ScreenWinTestSingleDisplay1x, GetNumDisplays) {
   EXPECT_EQ(1, GetScreen()->GetNumDisplays());
 }
 
 TEST_F(ScreenWinTestSingleDisplay1x, GetDisplayNearestWindowPrimaryDisplay) {
   Screen* screen = GetScreen();
   EXPECT_EQ(screen->GetPrimaryDisplay(),
             screen->GetDisplayNearestWindow(nullptr));
 }
 
@@ skipping 3233 matching lines @@
   // this test is to make sure we don't crash.
   ScreenWin::GetSystemMetricsInDIP(SM_CXSIZEFRAME);
 }
 
 TEST_F(ScreenWinUninitializedForced2x, GetScaleFactorForHWND) {
   EXPECT_EQ(2.0, ScreenWin::GetScaleFactorForHWND(nullptr));
 }
 
 }  // namespace win
 }  // namespace display