Revert of Add display::GetDisplayNearestView

Revert of Add display::GetDisplayNearestView (patchset #17 id:790001 of https://codereview.chromium.org/2688413007/ )

Reason for revert:
This CL causes a compile failure on a Mac bot:
https://build.chromium.org/p/chromium/builders/Mac/builds/24801

[38396/49613] CXX obj/extensions/browser/browser_tests/system_display_apitest.o
FAILED: obj/extensions/browser/browser_tests/system_display_apitest.o 
(snip)
../../extensions/browser/api/system_display/system_display_apitest.cc:58:37: error: non-virtual member function marked 'override' hides virtual member function
      gfx::NativeView window) const override {
                                    ^
../../ui/display/screen.h:61:19: note: hidden overloaded virtual function 'display::Screen::GetDisplayNearestWindow' declared here: type mismatch at 1st parameter ('gfx::NativeWindow' (aka 'NSWindow *') vs 'gfx::NativeView' (aka 'NSView *'))
  virtual Display GetDisplayNearestWindow(gfx::NativeWindow window) const = 0;
                  ^
../../extensions/browser/api/system_display/system_display_apitest.cc:200:61: error: allocating an object of abstract class type 'extensions::MockScreen'
      : provider_(new MockDisplayInfoProvider), screen_(new MockScreen) {}
                                                            ^
../../ui/display/screen.h:61:19: note: unimplemented pure virtual method 'GetDisplayNearestWindow' in 'MockScreen'
  virtual Display GetDisplayNearestWindow(gfx::NativeWindow window) const = 0;
                  ^
2 errors generated.

Original issue's description:
> Add display::GetDisplayNearestView
> 
> Add a method that gets the display with a given gfx::NativeView.
> The present API |display::GetDisplayNearestWindow| was also updated
> to accept gfx::NativeWindow to match its signature.
> 
> BUG=671401
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
> TBR=benwells@chromium.org,sky@chromium.org,boliu@chromium.org,msw@chromium.org,halliwell@chromium.org,bshe@chromium.org,alexclarke@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2688413007
> Cr-Commit-Position: refs/heads/master@{#457016}
> Committed: https://chromium.googlesource.com/chromium/src/+/93bb5a29073d2cb41b20d3cc6cf59459a3906043

TBR=oshima@chromium.org,boliu@chromium.org,tapted@chromium.org,benwells@chromium.org,sky@chromium.org,msw@chromium.org,halliwell@chromium.org,bshe@chromium.org,alexclarke@chromium.org,jinsukkim@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=671401
Review-Url: https://codereview.chromium.org/2747143005
Cr-Commit-Position: refs/heads/master@{#457025}
Committed: https://chromium.googlesource.com/chromium/src/+/a0c89e94b3310fa12b58db00eaca0e66c2f98c7b

[tzik, 2017-03-15 07:42:30]

The CQ bit was checked by tzik@chromium.org

[tzik, 2017-03-15 07:42:31]

Created Revert of Add display::GetDisplayNearestView

[commit-bot: I haz the power, 2017-03-15 07:42:44]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-15 07:43:59]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489563750405710, "parent_rev":
"8d9c69e2ce36013e57b553ce6ebdce48dcc2e522", "commit_rev":
"a0c89e94b3310fa12b58db00eaca0e66c2f98c7b"}

[commit-bot: I haz the power, 2017-03-15 07:44:45]

Description was changed from

==========
Revert of Add display::GetDisplayNearestView (patchset #17 id:790001 of
https://codereview.chromium.org/2688413007/ )

Reason for revert:
This CL causes a compile failure on a Mac bot:
https://build.chromium.org/p/chromium/builders/Mac/builds/24801

[38396/49613] CXX obj/extensions/browser/browser_tests/system_display_apitest.o
FAILED: obj/extensions/browser/browser_tests/system_display_apitest.o 
(snip)
../../extensions/browser/api/system_display/system_display_apitest.cc:58:37:
error: non-virtual member function marked 'override' hides virtual member
function
      gfx::NativeView window) const override {
                                    ^
../../ui/display/screen.h:61:19: note: hidden overloaded virtual function
'display::Screen::GetDisplayNearestWindow' declared here: type mismatch at 1st
parameter ('gfx::NativeWindow' (aka 'NSWindow *') vs 'gfx::NativeView' (aka
'NSView *'))
  virtual Display GetDisplayNearestWindow(gfx::NativeWindow window) const = 0;
                  ^
../../extensions/browser/api/system_display/system_display_apitest.cc:200:61:
error: allocating an object of abstract class type 'extensions::MockScreen'
      : provider_(new MockDisplayInfoProvider), screen_(new MockScreen) {}
                                                            ^
../../ui/display/screen.h:61:19: note: unimplemented pure virtual method
'GetDisplayNearestWindow' in 'MockScreen'
  virtual Display GetDisplayNearestWindow(gfx::NativeWindow window) const = 0;
                  ^
2 errors generated.

Original issue's description:
> Add display::GetDisplayNearestView
> 
> Add a method that gets the display with a given gfx::NativeView.
> The present API |display::GetDisplayNearestWindow| was also updated
> to accept gfx::NativeWindow to match its signature.
> 
> BUG=671401
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
>
TBR=benwells@chromium.org,sky@chromium.org,boliu@chromium.org,msw@chromium.or...
> 
> Review-Url: https://codereview.chromium.org/2688413007
> Cr-Commit-Position: refs/heads/master@{#457016}
> Committed:
https://chromium.googlesource.com/chromium/src/+/93bb5a29073d2cb41b20d3cc6cf5...

TBR=oshima@chromium.org,boliu@chromium.org,tapted@chromium.org,benwells@chrom...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=671401
==========

to

==========
Revert of Add display::GetDisplayNearestView (patchset #17 id:790001 of
https://codereview.chromium.org/2688413007/ )

Reason for revert:
This CL causes a compile failure on a Mac bot:
https://build.chromium.org/p/chromium/builders/Mac/builds/24801

[38396/49613] CXX obj/extensions/browser/browser_tests/system_display_apitest.o
FAILED: obj/extensions/browser/browser_tests/system_display_apitest.o 
(snip)
../../extensions/browser/api/system_display/system_display_apitest.cc:58:37:
error: non-virtual member function marked 'override' hides virtual member
function
      gfx::NativeView window) const override {
                                    ^
../../ui/display/screen.h:61:19: note: hidden overloaded virtual function
'display::Screen::GetDisplayNearestWindow' declared here: type mismatch at 1st
parameter ('gfx::NativeWindow' (aka 'NSWindow *') vs 'gfx::NativeView' (aka
'NSView *'))
  virtual Display GetDisplayNearestWindow(gfx::NativeWindow window) const = 0;
                  ^
../../extensions/browser/api/system_display/system_display_apitest.cc:200:61:
error: allocating an object of abstract class type 'extensions::MockScreen'
      : provider_(new MockDisplayInfoProvider), screen_(new MockScreen) {}
                                                            ^
../../ui/display/screen.h:61:19: note: unimplemented pure virtual method
'GetDisplayNearestWindow' in 'MockScreen'
  virtual Display GetDisplayNearestWindow(gfx::NativeWindow window) const = 0;
                  ^
2 errors generated.

Original issue's description:
> Add display::GetDisplayNearestView
> 
> Add a method that gets the display with a given gfx::NativeView.
> The present API |display::GetDisplayNearestWindow| was also updated
> to accept gfx::NativeWindow to match its signature.
> 
> BUG=671401
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
>
TBR=benwells@chromium.org,sky@chromium.org,boliu@chromium.org,msw@chromium.or...
> 
> Review-Url: https://codereview.chromium.org/2688413007
> Cr-Commit-Position: refs/heads/master@{#457016}
> Committed:
https://chromium.googlesource.com/chromium/src/+/93bb5a29073d2cb41b20d3cc6cf5...

TBR=oshima@chromium.org,boliu@chromium.org,tapted@chromium.org,benwells@chrom...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=671401

Review-Url: https://codereview.chromium.org/2747143005
Cr-Commit-Position: refs/heads/master@{#457025}
Committed:
https://chromium.googlesource.com/chromium/src/+/a0c89e94b3310fa12b58db00eaca...
==========

[commit-bot: I haz the power, 2017-03-15 07:44:46]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/a0c89e94b3310fa12b58db00eaca...