Add full touch support to mus.

services/ui/display/screen_manager_ozone.cc

Index: services/ui/display/screen_manager_ozone.cc
diff --git a/services/ui/display/screen_manager_ozone.cc b/services/ui/display/screen_manager_ozone.cc
index e48d6a1e78243bc3b6d3025a0ad727da5aacf7d0..a5a12927c495659f7d583fc596bc9bc8b0bfcb69 100644
--- a/services/ui/display/screen_manager_ozone.cc
+++ b/services/ui/display/screen_manager_ozone.cc
@@ -14,6 +14,7 @@
 #include "services/service_manager/public/cpp/interface_registry.h"
 #include "third_party/skia/include/core/SkColor.h"
 #include "ui/display/manager/chromeos/display_change_observer.h"
+#include "ui/display/manager/chromeos/touch_transform_controller.h"
 #include "ui/display/manager/display_layout_store.h"
 #include "ui/display/manager/display_manager_utilities.h"
 #include "ui/display/screen.h"
@@ -63,6 +64,9 @@ ScreenManagerOzone::~ScreenManagerOzone() {
   // We are shutting down and don't want to make anymore display changes.
   fake_display_controller_ = nullptr;
 
+  if (touch_transform_controller_)

[msw, 2017/01/10 19:43:27]

optional nit: check is not necessary

[kylechar, 2017/01/10 22:12:18]

Done.

+    touch_transform_controller_.reset();
+
   if (display_manager_)
     display_manager_->RemoveObserver(this);
 
@@ -180,6 +184,9 @@ void ScreenManagerOzone::Init(ScreenManagerDelegate* delegate) {
   // Perform initial configuration.
   display_configurator_.Init(std::move(native_display_delegate_), false);
   display_configurator_.ForceInitialConfigure(kChromeOsBootColor);
+
+  touch_transform_controller_ = base::MakeUnique<TouchTransformController>(

[msw, 2017/01/10 19:43:27]

I don't get results for TouchTransformController in codesearch or grepping my
local repo, and I don't know touch code well. Otherwise, your CL otherwise looks
fine; I just can't approve without some idea of what you're instantiating here
(and how it compares with the removed TouchController).

[kylechar, 2017/01/10 22:12:18]

The current class is ash::TouchTransformerController, it's renamed in the parent
CL. I'm assuming there used to be some class TouchTransformer that no longer
exists because there are class/method names that reference it. The code now just
uses gfx::Transform instead, so I changed the manager class name too.

TouchController was a hacky version of TouchTransformerController I wrote that
to get pixel touchscreen working.

[msw, 2017/01/10 22:36:57]

Acknowledged; thanks for pointing out the change dependency that I missed.

+      &display_configurator_, display_manager_.get());
 }
 
 void ScreenManagerOzone::RequestCloseDisplay(int64_t display_id) {
@@ -356,6 +363,9 @@ void ScreenManagerOzone::PostDisplayConfigurationChange(
     }
   }
 
+  if (touch_transform_controller_)

[msw, 2017/01/10 19:43:27]

q: does this ever happen before init or after reset in the dtor? (is this check
necessary?)

[kylechar, 2017/01/10 22:12:18]

It shouldn't, only if someone changed the order in the destructor, so it's
probably not needed. Done.

+    touch_transform_controller_->UpdateTouchTransforms();
+
   DVLOG(1) << "PostDisplayConfigurationChange";
 }