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

Issue 2158863002: Change initTouchEvent removal warning to M54 (Closed)

Created:
4 years, 5 months ago by chongz
Modified:
4 years, 5 months ago
Reviewers:
bokan
CC:
chromium-reviews, blink-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Change initTouchEvent removal warning to M54 This CL updates TouchEvent.InitTouchEvent removal warning to M54 since we missed M53 branch point. (The original plan was to remove in M53) The Chrome Status Entry has also been updated to M54: https://www.chromestatus.com/features/5730982598541312 BUG=629554 Committed: https://crrev.com/ae2c536166e766bdff9b4a7ccb8757cd9b569598 Cr-Commit-Position: refs/heads/master@{#406350}

Patch Set 1 : Change remove warning for initTouchEvent to M54 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M third_party/WebKit/LayoutTests/fast/events/touch/document-create-touch-list-expected.txt View 1 chunk +1 line, -1 line 0 comments Download
M third_party/WebKit/LayoutTests/fast/events/touch/touch-event-dispatch-no-crash-expected.txt View 1 chunk +1 line, -1 line 0 comments Download
M third_party/WebKit/Source/core/frame/Deprecation.cpp View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 24 (18 generated)
chongz
bokan@ can you take a look at this CL please? I'm planning to merge it ...
4 years, 5 months ago (2016-07-18 18:48:07 UTC) #13
bokan
Please wrap the description at 80 cols Otherwise, lgtm
4 years, 5 months ago (2016-07-19 16:11:18 UTC) #14
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2158863002/20001
4 years, 5 months ago (2016-07-19 17:47:37 UTC) #19
commit-bot: I haz the power
Committed patchset #1 (id:20001)
4 years, 5 months ago (2016-07-19 19:43:18 UTC) #21
commit-bot: I haz the power
CQ bit was unchecked.
4 years, 5 months ago (2016-07-19 19:43:21 UTC) #22
commit-bot: I haz the power
4 years, 5 months ago (2016-07-19 19:45:10 UTC) #24
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/ae2c536166e766bdff9b4a7ccb8757cd9b569598
Cr-Commit-Position: refs/heads/master@{#406350}

Powered by Google App Engine
This is Rietveld 408576698