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

Unified Diff: chrome/browser/extensions/display_info_provider_aura.h

Issue 476103002: Make DisplayInfoProvider an interface (Closed) Base URL: git@github.com:tmpsantos/chromium.git@display_info
Patch Set: Addressed review comments. Created 6 years, 4 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/display_info_provider_aura.h
diff --git a/chrome/browser/extensions/display_info_provider_aura.h b/chrome/browser/extensions/display_info_provider_aura.h
new file mode 100644
index 0000000000000000000000000000000000000000..41171dda60235041406e72932310d3e98adf6297
--- /dev/null
+++ b/chrome/browser/extensions/display_info_provider_aura.h
@@ -0,0 +1,33 @@
+// Copyright 2014 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.
+
+#ifndef CHROME_BROWSER_EXTENSIONS_DISPLAY_INFO_PROVIDER_AURA_H_
+#define CHROME_BROWSER_EXTENSIONS_DISPLAY_INFO_PROVIDER_AURA_H_
+
+#include <string>
+
+#include "chrome/browser/extensions/api/system_display/display_info_provider.h"
+
+namespace extensions {
+
+class DisplayInfoProviderAura : public DisplayInfoProvider {
+ public:
+ DisplayInfoProviderAura();
+ virtual ~DisplayInfoProviderAura();
+
+ virtual bool SetInfo(const std::string& display_id,
James Cook 2014/08/19 21:48:51 nit: put a comment like "// DisplayInfoProvider ov
tmpsantos 2014/08/20 18:58:31 Done on all platform specific headers.
+ const api::system_display::DisplayProperties& info,
+ std::string* error) OVERRIDE;
+
James Cook 2014/08/19 21:48:51 nit: no blank line
tmpsantos 2014/08/20 18:58:31 Done on all platform specific headers.
+ virtual void UpdateDisplayUnitInfoForPlatform(
+ const gfx::Display& display,
+ extensions::api::system_display::DisplayUnitInfo* unit) OVERRIDE;
+
+ private:
+ DISALLOW_COPY_AND_ASSIGN(DisplayInfoProviderAura);
+};
+
+} // namespace extensions
+
+#endif // CHROME_BROWSER_EXTENSIONS_DISPLAY_INFO_PROVIDER_AURA_H_

Powered by Google App Engine
This is Rietveld 408576698