New issue
Advanced search Search tips

Issue 705686 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: May 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocking:
issue 706589



Sign in to add a comment

Get AshTouchTransformController working for mash/mushrome

Project Member Reported by sky@chromium.org, Mar 27 2017

Issue description

AshTouchTransformController calls to UpdateTouchTransforms() at a couple of points that relies on the WindowTreeHostManager. I don't think it makes sense for  ash to use display::TouchTransformController at all in mus/mushrome. So, either this code can safely be ignored for mus/mushrome, or we need to make mus do what AshTouchTransformController is doing.
 
Status: Available (was: Untriaged)
We use TouchTransformController in mus already. ScreenManager calls TouchTransformController::UpdateTouchTransforms() when the display configuration changes. This is the same logic that AshTouchTransformController adds for cash.

The only thing we need to do in mus/mushrome is make sure to not create a AshTouchTransformController.

Comment 2 by sky@chromium.org, May 18 2017

Blocking: 706589
Owner: sky@chromium.org
Status: Started (was: Available)
Project Member

Comment 3 by bugdroid1@chromium.org, May 21 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/2f68b6fe7c3249c69aa561addd4a54b257e2d2a4

commit 2f68b6fe7c3249c69aa561addd4a54b257e2d2a4
Author: sky <sky@chromium.org>
Date: Sun May 21 16:08:39 2017

chromeos: moves setting of touch state to a separate class

For mash/mushrome we need setting of touch state to happen in
mus. This patch separates out setting into a separate class so that
ash in mus/mash can plug in a class that makes mojo calls.

BUG= 705686 
TEST=covered by tests
R=oshima@chromium.org

Review-Url: https://codereview.chromium.org/2887413004
Cr-Commit-Position: refs/heads/master@{#473474}

[modify] https://crrev.com/2f68b6fe7c3249c69aa561addd4a54b257e2d2a4/ash/shell.cc
[modify] https://crrev.com/2f68b6fe7c3249c69aa561addd4a54b257e2d2a4/ash/touch/ash_touch_transform_controller.cc
[modify] https://crrev.com/2f68b6fe7c3249c69aa561addd4a54b257e2d2a4/ash/touch/ash_touch_transform_controller.h
[modify] https://crrev.com/2f68b6fe7c3249c69aa561addd4a54b257e2d2a4/services/ui/display/screen_manager_ozone_internal.cc
[modify] https://crrev.com/2f68b6fe7c3249c69aa561addd4a54b257e2d2a4/ui/display/manager/BUILD.gn
[modify] https://crrev.com/2f68b6fe7c3249c69aa561addd4a54b257e2d2a4/ui/display/manager/chromeos/DEPS
[add] https://crrev.com/2f68b6fe7c3249c69aa561addd4a54b257e2d2a4/ui/display/manager/chromeos/default_touch_transform_setter.cc
[add] https://crrev.com/2f68b6fe7c3249c69aa561addd4a54b257e2d2a4/ui/display/manager/chromeos/default_touch_transform_setter.h
[add] https://crrev.com/2f68b6fe7c3249c69aa561addd4a54b257e2d2a4/ui/display/manager/chromeos/touch_device_transform.cc
[add] https://crrev.com/2f68b6fe7c3249c69aa561addd4a54b257e2d2a4/ui/display/manager/chromeos/touch_device_transform.h
[modify] https://crrev.com/2f68b6fe7c3249c69aa561addd4a54b257e2d2a4/ui/display/manager/chromeos/touch_transform_controller.cc
[modify] https://crrev.com/2f68b6fe7c3249c69aa561addd4a54b257e2d2a4/ui/display/manager/chromeos/touch_transform_controller.h
[modify] https://crrev.com/2f68b6fe7c3249c69aa561addd4a54b257e2d2a4/ui/display/manager/chromeos/touch_transform_controller_unittest.cc
[add] https://crrev.com/2f68b6fe7c3249c69aa561addd4a54b257e2d2a4/ui/display/manager/chromeos/touch_transform_setter.h

Project Member

Comment 4 by bugdroid1@chromium.org, May 22 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/68e20f488cc2ea0216832c8fe3640d9b748595d2

commit 68e20f488cc2ea0216832c8fe3640d9b748595d2
Author: sky <sky@chromium.org>
Date: Mon May 22 19:17:33 2017

chromeos: adds TouchDeviceServer and wires up in mushrome

TouchDeviceServer is necessary for hooking up AshTouchTransformController in mus/mash.

BUG= 705686 
TEST=covered by tests
R=erg@chromium.org,rockot@chromium.org,tsepez@chromium.org

Review-Url: https://codereview.chromium.org/2901563002
Cr-Commit-Position: refs/heads/master@{#473651}

[modify] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/ash/aura/shell_port_classic.cc
[modify] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/ash/aura/shell_port_classic.h
[modify] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/ash/mus/BUILD.gn
[modify] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/ash/mus/bridge/shell_port_mash.cc
[modify] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/ash/mus/bridge/shell_port_mash.h
[add] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/ash/mus/touch_transform_setter_mus.cc
[add] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/ash/mus/touch_transform_setter_mus.h
[modify] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/ash/shell.cc
[modify] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/ash/shell.h
[modify] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/ash/shell_port.h
[modify] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/mojo/public/tools/bindings/chromium_bindings_configuration.gni
[modify] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/mojo/public/tools/bindings/mojom.gni
[modify] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/services/ui/input_devices/BUILD.gn
[modify] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/services/ui/input_devices/input_device_server.cc
[modify] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/services/ui/input_devices/input_device_server.h
[add] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/services/ui/input_devices/touch_device_server.cc
[add] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/services/ui/input_devices/touch_device_server.h
[modify] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/services/ui/manifest.json
[modify] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/services/ui/public/interfaces/input_devices/BUILD.gn
[add] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/services/ui/public/interfaces/input_devices/touch_device_server.mojom
[modify] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/ui/display/BUILD.gn
[add] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/ui/display/manager/chromeos/mojo/BUILD.gn
[add] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/ui/display/manager/chromeos/mojo/OWNERS
[add] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/ui/display/manager/chromeos/mojo/touch_device_transform.mojom
[add] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/ui/display/manager/chromeos/mojo/touch_device_transform.typemap
[add] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/ui/display/manager/chromeos/mojo/touch_device_transform_struct_traits.h
[add] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/ui/display/manager/chromeos/mojo/touch_device_transform_struct_traits_unittest.cc
[add] https://crrev.com/68e20f488cc2ea0216832c8fe3640d9b748595d2/ui/display/manager/chromeos/mojo/typemaps.gni

Project Member

Comment 5 by bugdroid1@chromium.org, May 22 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/d99444dbb09468ef53fc699708c0b68afb451baa

commit d99444dbb09468ef53fc699708c0b68afb451baa
Author: mek <mek@chromium.org>
Date: Mon May 22 19:50:48 2017

Revert of chromeos: adds TouchDeviceServer and wires up in mushrome (patchset #1 id:1 of https://codereview.chromium.org/2901563002/ )

Reason for revert:
Failed compile on https://build.chromium.org/p/chromium.chromiumos/builders/ChromiumOS%20x86-generic%20Compile/builds/35937:

/b/c/goma_client/gomacc i686-pc-linux-gnu-g++ -B/b/c/b/ChromiumOS_x86_generic_Compile/.cros_cache/chrome-sdk/tarballs/x86-generic+9536.0.0+target_toolchain/usr/x86_64-pc-linux-gnu/i686-pc-linux-gnu/binutils-bin/2.25.51-gold -MMD -MF obj/ash/ash/shell_port_classic.o.d -DASH_IMPLEMENTATION -DV8_DEPRECATION_WARNINGS -DUSE_UDEV -DUSE_ASH=1 -DUSE_AURA=1 -DUSE_NSS_CERTS=1 -DUSE_OZONE=1 -DFULL_SAFE_BROWSING -DSAFE_BROWSING_CSD -DSAFE_BROWSING_DB_LOCAL -DCHROMIUM_BUILD -DFIELDTRIAL_TESTING_ENABLED -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D_FORTIFY_SOURCE=2 -DOS_CHROMEOS -DNDEBUG -DNVALGRIND -DDYNAMIC_ANNOTATIONS_ENABLED=0 -DUSE_EGL -DTOOLKIT_VIEWS=1 -DU_USING_ICU_NAMESPACE=0 -DU_ENABLE_DYLOAD=0 -DU_STATIC_IMPLEMENTATION -DICU_UTIL_DATA_IMPL=ICU_UTIL_DATA_FILE -DUCHAR_TYPE=uint16_t -DSK_IGNORE_LINEONLY_AA_CONVEX_PATH_OPTS -DSK_HAS_PNG_LIBRARY -DSK_HAS_WEBP_LIBRARY -DSK_HAS_JPEG_LIBRARY -DSK_SUPPORT_GPU=1 -DGOOGLE_PROTOBUF_NO_RTTI -DGOOGLE_PROTOBUF_NO_STATIC_INITIALIZER -DHAVE_PTHREAD -DMESA_EGL_NO_X11_HEADERS -I../.. -Igen -I../../third_party/libwebp -I../../third_party/khronos -I../../gpu -I../../third_party/ced/src -I../../third_party/icu/source/common -I../../third_party/icu/source/i18n -I../../skia/config -I../../skia/ext -I../../third_party/skia/include/c -I../../third_party/skia/include/config -I../../third_party/skia/include/core -I../../third_party/skia/include/effects -I../../third_party/skia/include/encode -I../../third_party/skia/include/images -I../../third_party/skia/include/lazy -I../../third_party/skia/include/pathops -I../../third_party/skia/include/pdf -I../../third_party/skia/include/pipe -I../../third_party/skia/include/ports -I../../third_party/skia/include/utils -I../../third_party/skia/third_party/vulkan -I../../third_party/skia/include/gpu -I../../third_party/skia/src/gpu -I../../third_party/skia/src/sksl -I/b/c/b/ChromiumOS_x86_generic_Compile/.cros_cache/chrome-sdk/tarballs/x86-generic+9536.0.0+sysroot_chromeos-base_chromeos-chrome.tar.xz/usr/include/dbus-1.0 -I/b/c/b/ChromiumOS_x86_generic_Compile/.cros_cache/chrome-sdk/tarballs/x86-generic+9536.0.0+sysroot_chromeos-base_chromeos-chrome.tar.xz/usr/lib/dbus-1.0/include -I../../third_party/protobuf/src -Igen/protoc_out -I../../third_party/protobuf/src -I../../third_party/libwebm/source -I../../third_party/boringssl/src/include -I/b/c/b/ChromiumOS_x86_generic_Compile/.cros_cache/chrome-sdk/tarballs/x86-generic+9536.0.0+sysroot_chromeos-base_chromeos-chrome.tar.xz/usr/include/nss -I/b/c/b/ChromiumOS_x86_generic_Compile/.cros_cache/chrome-sdk/tarballs/x86-generic+9536.0.0+sysroot_chromeos-base_chromeos-chrome.tar.xz/usr/include/nspr -I../../third_party/WebKit -Igen/third_party/WebKit -I../../v8/include -Igen/v8/include -I../../third_party/qcms/src -Igen -I../../third_party/mesa/src/include -fno-strict-aliasing -Wno-builtin-macro-redefined -D__DATE__= -D__TIME__= -D__TIMESTAMP__= -funwind-tables -fPIC -pipe -B../../third_party/binutils/Linux_x64/Release/bin -m32 -msse2 -mfpmath=sse -mmmx -pthread -Wall -Werror -Wno-unused-local-typedefs -Wno-maybe-uninitialized -Wno-missing-field-initializers -Wno-unused-parameter -O2 -fno-ident -fdata-sections -ffunction-sections -fno-omit-frame-pointer -g2 -gsplit-dwarf --sysroot=../../../.cros_cache/chrome-sdk/tarballs/x86-generic+9536.0.0+sysroot_chromeos-base_chromeos-chrome.tar.xz -fvisibility=hidden -std=gnu++11 -Wno-narrowing -fno-rtti -fno-exceptions -fvisibility-inlines-hidden -march=i686 -pipe -march=i686 -pipe -pipe -march=i686 -mfpmath=sse -mmmx -msse -msse2 -msse3 -D__google_stl_debug_vector=1 -c ../../ash/aura/shell_port_classic.cc -o obj/ash/ash/shell_port_classic.o
../../ash/aura/shell_port_classic.cc: In member function 'virtual std::unique_ptr<display::TouchTransformSetter> ash::ShellPortClassic::CreateTouchTransformDelegate()':
../../ash/aura/shell_port_classic.cc:126:10: error: cannot bind 'std::unique_ptr<display::DefaultTouchTransformSetter>' lvalue to 'std::unique_ptr<display::DefaultTouchTransformSetter>&&'
   return delegate;
          ^
In file included from /b/c/b/ChromiumOS_x86_generic_Compile/.cros_cache/chrome-sdk/tarballs/x86-generic+9536.0.0+target_toolchain/usr/x86_64-pc-linux-gnu/i686-pc-linux-gnu/gcc-bin/4.9.x/../../../../lib/gcc/i686-pc-linux-gnu/4.9.x/include/g++-v4/memory:81:0,
                 from ../../ash/aura/shell_port_classic.h:8,
                 from ../../ash/aura/shell_port_classic.cc:5:
/b/c/b/ChromiumOS_x86_generic_Compile/.cros_cache/chrome-sdk/tarballs/x86-generic+9536.0.0+target_toolchain/usr/x86_64-pc-linux-gnu/i686-pc-linux-gnu/gcc-bin/4.9.x/../../../../lib/gcc/i686-pc-linux-gnu/4.9.x/include/g++-v4/bits/unique_ptr.h:220:2: note: initializing argument 1 of 'std::unique_ptr<_Tp, _Dp>::unique_ptr(std::unique_ptr<_Up, _Ep>&&) [with _Up = display::DefaultTouchTransformSetter; _Ep = std::default_delete<display::DefaultTouchTransformSetter>; <template-parameter-2-3> = void; _Tp = display::TouchTransformSetter; _Dp = std::default_delete<display::TouchTransformSetter>]'
  unique_ptr(unique_ptr<_Up, _Ep>&& __u) noexcept
  ^
../../ash/aura/shell_port_classic.cc:127:1: error: control reaches end of non-void function [-Werror=return-type]
 }
 ^

Original issue's description:
> chromeos: adds TouchDeviceServer and wires up in mushrome
>
> TouchDeviceServer is necessary for hooking up AshTouchTransformController in mus/mash.
>
> BUG= 705686 
> TEST=covered by tests
> R=erg@chromium.org,rockot@chromium.org,tsepez@chromium.org
>
> Review-Url: https://codereview.chromium.org/2901563002
> Cr-Commit-Position: refs/heads/master@{#473651}
> Committed: https://chromium.googlesource.com/chromium/src/+/68e20f488cc2ea0216832c8fe3640d9b748595d2

TBR=rockot@chromium.org,erg@chromium.org,tsepez@chromium.org,sky@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 705686 

Review-Url: https://codereview.chromium.org/2897863004
Cr-Commit-Position: refs/heads/master@{#473662}

[modify] https://crrev.com/d99444dbb09468ef53fc699708c0b68afb451baa/ash/aura/shell_port_classic.cc
[modify] https://crrev.com/d99444dbb09468ef53fc699708c0b68afb451baa/ash/aura/shell_port_classic.h
[modify] https://crrev.com/d99444dbb09468ef53fc699708c0b68afb451baa/ash/mus/BUILD.gn
[modify] https://crrev.com/d99444dbb09468ef53fc699708c0b68afb451baa/ash/mus/bridge/shell_port_mash.cc
[modify] https://crrev.com/d99444dbb09468ef53fc699708c0b68afb451baa/ash/mus/bridge/shell_port_mash.h
[delete] https://crrev.com/961a33a21ebca4823dd6e2578bdcb70013b6e4fc/ash/mus/touch_transform_setter_mus.cc
[delete] https://crrev.com/961a33a21ebca4823dd6e2578bdcb70013b6e4fc/ash/mus/touch_transform_setter_mus.h
[modify] https://crrev.com/d99444dbb09468ef53fc699708c0b68afb451baa/ash/shell.cc
[modify] https://crrev.com/d99444dbb09468ef53fc699708c0b68afb451baa/ash/shell.h
[modify] https://crrev.com/d99444dbb09468ef53fc699708c0b68afb451baa/ash/shell_port.h
[modify] https://crrev.com/d99444dbb09468ef53fc699708c0b68afb451baa/mojo/public/tools/bindings/chromium_bindings_configuration.gni
[modify] https://crrev.com/d99444dbb09468ef53fc699708c0b68afb451baa/mojo/public/tools/bindings/mojom.gni
[modify] https://crrev.com/d99444dbb09468ef53fc699708c0b68afb451baa/services/ui/input_devices/BUILD.gn
[modify] https://crrev.com/d99444dbb09468ef53fc699708c0b68afb451baa/services/ui/input_devices/input_device_server.cc
[modify] https://crrev.com/d99444dbb09468ef53fc699708c0b68afb451baa/services/ui/input_devices/input_device_server.h
[delete] https://crrev.com/961a33a21ebca4823dd6e2578bdcb70013b6e4fc/services/ui/input_devices/touch_device_server.cc
[delete] https://crrev.com/961a33a21ebca4823dd6e2578bdcb70013b6e4fc/services/ui/input_devices/touch_device_server.h
[modify] https://crrev.com/d99444dbb09468ef53fc699708c0b68afb451baa/services/ui/manifest.json
[modify] https://crrev.com/d99444dbb09468ef53fc699708c0b68afb451baa/services/ui/public/interfaces/input_devices/BUILD.gn
[delete] https://crrev.com/961a33a21ebca4823dd6e2578bdcb70013b6e4fc/services/ui/public/interfaces/input_devices/touch_device_server.mojom
[modify] https://crrev.com/d99444dbb09468ef53fc699708c0b68afb451baa/ui/display/BUILD.gn
[delete] https://crrev.com/961a33a21ebca4823dd6e2578bdcb70013b6e4fc/ui/display/manager/chromeos/mojo/BUILD.gn
[delete] https://crrev.com/961a33a21ebca4823dd6e2578bdcb70013b6e4fc/ui/display/manager/chromeos/mojo/OWNERS
[delete] https://crrev.com/961a33a21ebca4823dd6e2578bdcb70013b6e4fc/ui/display/manager/chromeos/mojo/touch_device_transform.mojom
[delete] https://crrev.com/961a33a21ebca4823dd6e2578bdcb70013b6e4fc/ui/display/manager/chromeos/mojo/touch_device_transform.typemap
[delete] https://crrev.com/961a33a21ebca4823dd6e2578bdcb70013b6e4fc/ui/display/manager/chromeos/mojo/touch_device_transform_struct_traits.h
[delete] https://crrev.com/961a33a21ebca4823dd6e2578bdcb70013b6e4fc/ui/display/manager/chromeos/mojo/touch_device_transform_struct_traits_unittest.cc
[delete] https://crrev.com/961a33a21ebca4823dd6e2578bdcb70013b6e4fc/ui/display/manager/chromeos/mojo/typemaps.gni

Project Member

Comment 6 by bugdroid1@chromium.org, May 22 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3

commit 7265b1cc285ca137198ba1f2d75bfc649aa2c3a3
Author: sky <sky@chromium.org>
Date: Mon May 22 23:11:03 2017

chromeos: adds TouchDeviceServer and wires up in mushrome

TouchDeviceServer is necessary for hooking up AshTouchTransformController in mus/mash.

BUG= 705686 
TEST=covered by tests
R=erg@chromium.org,rockot@chromium.org,tsepez@chromium.org

Review-Url: https://codereview.chromium.org/2901563002
Cr-Original-Commit-Position: refs/heads/master@{#473651}
Committed: https://chromium.googlesource.com/chromium/src/+/68e20f488cc2ea0216832c8fe3640d9b748595d2
Review-Url: https://codereview.chromium.org/2901563002
Cr-Commit-Position: refs/heads/master@{#473734}

[modify] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/ash/aura/shell_port_classic.cc
[modify] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/ash/aura/shell_port_classic.h
[modify] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/ash/mus/BUILD.gn
[modify] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/ash/mus/bridge/shell_port_mash.cc
[modify] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/ash/mus/bridge/shell_port_mash.h
[add] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/ash/mus/touch_transform_setter_mus.cc
[add] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/ash/mus/touch_transform_setter_mus.h
[modify] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/ash/shell.cc
[modify] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/ash/shell.h
[modify] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/ash/shell_port.h
[modify] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/mojo/public/tools/bindings/chromium_bindings_configuration.gni
[modify] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/mojo/public/tools/bindings/mojom.gni
[modify] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/services/ui/input_devices/BUILD.gn
[modify] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/services/ui/input_devices/input_device_server.cc
[modify] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/services/ui/input_devices/input_device_server.h
[add] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/services/ui/input_devices/touch_device_server.cc
[add] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/services/ui/input_devices/touch_device_server.h
[modify] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/services/ui/manifest.json
[modify] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/services/ui/public/interfaces/input_devices/BUILD.gn
[add] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/services/ui/public/interfaces/input_devices/touch_device_server.mojom
[modify] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/ui/display/BUILD.gn
[add] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/ui/display/manager/chromeos/mojo/BUILD.gn
[add] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/ui/display/manager/chromeos/mojo/OWNERS
[add] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/ui/display/manager/chromeos/mojo/touch_device_transform.mojom
[add] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/ui/display/manager/chromeos/mojo/touch_device_transform.typemap
[add] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/ui/display/manager/chromeos/mojo/touch_device_transform_struct_traits.h
[add] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/ui/display/manager/chromeos/mojo/touch_device_transform_struct_traits_unittest.cc
[add] https://crrev.com/7265b1cc285ca137198ba1f2d75bfc649aa2c3a3/ui/display/manager/chromeos/mojo/typemaps.gni

Comment 7 by sky@chromium.org, May 23 2017

Status: Fixed (was: Started)

Comment 8 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

Comment 9 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment