Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(421)

Side by Side Diff: chrome/browser/extensions/display_info_provider_mac.cc

Issue 1608053002: Remove now-unnecessary wrappers around gfx::Screen::GetScreen() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@metro-mode-5
Patch Set: . Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/extensions/display_info_provider_mac.h" 5 #include "chrome/browser/extensions/display_info_provider_mac.h"
6 6
7 #include "ui/gfx/screen.h" 7 #include "ui/gfx/screen.h"
oshima 2016/01/20 06:40:12 remove this?
scottmg 2016/01/26 20:46:35 Done.
8 8
9 namespace extensions { 9 namespace extensions {
10 10
11 DisplayInfoProviderMac::DisplayInfoProviderMac() { 11 DisplayInfoProviderMac::DisplayInfoProviderMac() {
12 } 12 }
13 13
14 DisplayInfoProviderMac::~DisplayInfoProviderMac() { 14 DisplayInfoProviderMac::~DisplayInfoProviderMac() {
15 } 15 }
16 16
17 bool DisplayInfoProviderMac::SetInfo( 17 bool DisplayInfoProviderMac::SetInfo(
18 const std::string& display_id, 18 const std::string& display_id,
19 const api::system_display::DisplayProperties& info, 19 const api::system_display::DisplayProperties& info,
20 std::string* error) { 20 std::string* error) {
21 *error = "Not implemented"; 21 *error = "Not implemented";
22 return false; 22 return false;
23 } 23 }
24 24
25 void DisplayInfoProviderMac::UpdateDisplayUnitInfoForPlatform( 25 void DisplayInfoProviderMac::UpdateDisplayUnitInfoForPlatform(
26 const gfx::Display& display, 26 const gfx::Display& display,
27 extensions::api::system_display::DisplayUnitInfo* unit) { 27 extensions::api::system_display::DisplayUnitInfo* unit) {
28 static bool logged_once = false; 28 static bool logged_once = false;
29 if (!logged_once) { 29 if (!logged_once) {
30 NOTIMPLEMENTED(); 30 NOTIMPLEMENTED();
31 logged_once = true; 31 logged_once = true;
32 } 32 }
33 } 33 }
34 34
35 gfx::Screen* DisplayInfoProviderMac::GetActiveScreen() {
36 return gfx::Screen::GetScreen();
37 }
38
39 // static 35 // static
40 DisplayInfoProvider* DisplayInfoProvider::Create() { 36 DisplayInfoProvider* DisplayInfoProvider::Create() {
41 return new DisplayInfoProviderMac(); 37 return new DisplayInfoProviderMac();
42 } 38 }
43 39
44 } // namespace extensions 40 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698