New issue
Advanced search Search tips

Issue 625000 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2017
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Bug



Sign in to add a comment

Cleanup after crrev.com/2111593003: move WebInputEventTraitsTest's "[Can]Coalesce" tests into the EventWithLatencyInfoTest test harness

Project Member Reported by tapted@chromium.org, Jul 1 2016

Issue description

Chrome Version       : 53.0.2774.3

https://codereview.chromium.org/2111593003/ decouples WebInputEventTraits and EventWithLatencyInfo. Most of WebInputEventTraitsTest is for the stuff used by EventWithLatencyInfo, so they should move to EventWithLatencyInfoTest
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 4 2016

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

commit 63119c20ac70ec63df097d0bf0901b4ad0d2ae2e
Author: tapted <tapted@chromium.org>
Date: Mon Jul 04 02:05:09 2016

Consolidate WebInputEvent Coalescing unit tests

They were previously split across EventWithLatencyInfo and
WebInputEventTraits tests, but EventWithLatencyInfo was decoupled from
WebInputEventTraits in r403402.

BUG= 625000 

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

[modify] https://crrev.com/63119c20ac70ec63df097d0bf0901b4ad0d2ae2e/content/common/input/event_with_latency_info_unittest.cc
[modify] https://crrev.com/63119c20ac70ec63df097d0bf0901b4ad0d2ae2e/content/common/input/web_input_event_traits_unittest.cc

Comment 2 by tapted@chromium.org, Jan 27 2017

Status: Fixed (was: Assigned)

Sign in to add a comment