Make DisplayInfoProvider an interface

chrome/browser/extensions/display_info_provider_chromeos.h

+// 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_CHROMEOS_H_
+#define CHROME_BROWSER_EXTENSIONS_DISPLAY_INFO_PROVIDER_CHROMEOS_H_
+
+#include <string>

[James Cook, 2014/08/19 21:48:52]

nit: you don't need this, generally for types that are included as part of an
interface we don't re-include the header

[tmpsantos, 2014/08/20 18:58:31]

I originally wrote without it, but "git cl lint" was complaining.

chrome/browser/extensions/display_info_provider_chromeos.h:20:  Add #include
<string> for string  [build/include_what_you_use] [4]

I'm assuming as a false positive and removing this from all the platform
specific hearders.

+
+#include "chrome/browser/extensions/api/system_display/display_info_provider.h"
+
+namespace extensions {
+
+class DisplayInfoProviderChromeOS : public DisplayInfoProvider {
+ public:
+  DisplayInfoProviderChromeOS();
+  virtual ~DisplayInfoProviderChromeOS();
+
+  virtual bool SetInfo(const std::string& display_id,
+                       const api::system_display::DisplayProperties& info,
+                       std::string* error) OVERRIDE;
+
+  virtual void UpdateDisplayUnitInfoForPlatform(
+      const gfx::Display& display,
+      extensions::api::system_display::DisplayUnitInfo* unit) OVERRIDE;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(DisplayInfoProviderChromeOS);
+};
+
+}  // namespace extensions
+
+#endif  // CHROME_BROWSER_EXTENSIONS_DISPLAY_INFO_PROVIDER_CHROMEOS_H_