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

Unified Diff: ui/display/chromeos/display_configurator.h

Issue 336863002: Moving input device hotplug event processing outside of ui/display (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 6 years, 3 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: ui/display/chromeos/display_configurator.h
diff --git a/ui/display/chromeos/display_configurator.h b/ui/display/chromeos/display_configurator.h
index cae408670a2f078c93b4935a5dee5ad82da12cb2..30e25b850b41f7937dc340c8cdb604dee2206100 100644
--- a/ui/display/chromeos/display_configurator.h
+++ b/ui/display/chromeos/display_configurator.h
@@ -42,9 +42,6 @@ class DISPLAY_EXPORT DisplayConfigurator : public NativeDisplayObserver {
DisplaySnapshot* display; // Not owned.
- // XInput device ID or 0 if this display isn't a touchscreen.
- int touch_device_id;
-
// User-selected mode for the display.
const DisplayMode* selected_mode;
@@ -97,18 +94,6 @@ class DISPLAY_EXPORT DisplayConfigurator : public NativeDisplayObserver {
virtual bool SoftwareMirroringEnabled() const = 0;
};
- class TouchscreenDelegate {
- public:
- virtual ~TouchscreenDelegate() {}
-
- // Searches for touchscreens among input devices,
- // and tries to match them up to screens in |displays|.
- // |displays| is an array of detected screens.
- // If a touchscreen with same resolution as a display's native mode
- // is detected, its id will be stored in this display.
- virtual void AssociateTouchscreens(std::vector<DisplayState>* displays) = 0;
- };
-
// Helper class used by tests.
class TestApi {
public:
@@ -165,12 +150,10 @@ class DISPLAY_EXPORT DisplayConfigurator : public NativeDisplayObserver {
mirroring_controller_ = controller;
}
- // Replaces |native_display_delegate_| and |touchscreen_delegate_| with the 2
- // delegates passed in and sets |configure_display_| to true. Should be called
- // before Init().
- void SetDelegatesForTesting(
- scoped_ptr<NativeDisplayDelegate> display_delegate,
- scoped_ptr<TouchscreenDelegate> touchscreen_delegate);
+ // Replaces |native_display_delegate_| with the delegate passed in and sets
+ // |configure_display_| to true. Should be called before Init().
+ void SetDelegateForTesting(
+ scoped_ptr<NativeDisplayDelegate> display_delegate);
// Sets the initial value of |power_state_|. Must be called before Start().
void SetInitialDisplayPower(chromeos::DisplayPowerState power_state);
@@ -321,7 +304,6 @@ class DISPLAY_EXPORT DisplayConfigurator : public NativeDisplayObserver {
StateController* state_controller_;
SoftwareMirroringController* mirroring_controller_;
scoped_ptr<NativeDisplayDelegate> native_display_delegate_;
- scoped_ptr<TouchscreenDelegate> touchscreen_delegate_;
// Used to enable modes which rely on panel fitting.
bool is_panel_fitting_enabled_;

Powered by Google App Engine
This is Rietveld 408576698