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

Issue 1553263002: Remove Pass on CrOS. (Closed)

Created:
4 years, 11 months ago by dcheng
Modified:
4 years, 11 months ago
CC:
chromium-reviews, gavinp+memory_chromium.org, vmpstr+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Remove Pass on CrOS. Pass() is a legacy of the original C++03 move emulation implementation. New code should use std::move from <utility>. BUG=557422 R=danakj@chromium.org,sandersd@chromium.org,spang@chromium.org TBR=reveman@chromium.org,sky@chromium.org Committed: https://crrev.com/78820a435f2d94fb08e9a4cdb810b811c6cbdea9 Cr-Commit-Position: refs/heads/master@{#368009}

Patch Set 1 #

Patch Set 2 : Also include a change that lives behind a gyp define #

Patch Set 3 : Blind fixes in //ui #

Patch Set 4 : Ozone fixes in //ash #

Patch Set 5 : Another exo fix #

Patch Set 6 : Fix //components/exo too #

Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -11 lines) Patch
M ash/host/ash_window_tree_host_ozone.cc View 1 2 3 2 chunks +3 lines, -1 line 0 comments Download
M base/memory/scoped_ptr_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M base/move.h View 1 chunk +1 line, -1 line 0 comments Download
M components/exo/display.cc View 1 2 3 4 3 chunks +4 lines, -2 lines 0 comments Download
M components/exo/wayland/server.cc View 1 2 3 4 5 1 chunk +1 line, -1 line 0 comments Download
M content/common/gpu/media/v4l2_video_encode_accelerator.cc View 1 3 chunks +3 lines, -2 lines 0 comments Download
M ui/events/ozone/evdev/input_device_factory_evdev.cc View 1 2 2 chunks +2 lines, -2 lines 0 comments Download
M ui/events/ozone/evdev/libgestures_glue/event_reader_libevdev_cros.cc View 1 2 2 chunks +2 lines, -1 line 0 comments Download

Messages

Total messages: 46 (24 generated)
dcheng
Worked on my machine. Let's ship it!
4 years, 11 months ago (2016-01-04 22:37:24 UTC) #1
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1553263002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1553263002/1
4 years, 11 months ago (2016-01-04 22:40:15 UTC) #3
danakj
LGTM
4 years, 11 months ago (2016-01-04 22:43:12 UTC) #4
commit-bot: I haz the power
Dry run: Try jobs failed on following builders: chromeos_amd64-generic_chromium_compile_only_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-generic_chromium_compile_only_ng/builds/75104) chromeos_x86-generic_chromium_compile_only_ng on ...
4 years, 11 months ago (2016-01-04 23:12:18 UTC) #6
dcheng
TBRing sandersd@ for //content/common/gpu/media changes (which are behind a GN/GYP flag)
4 years, 11 months ago (2016-01-04 23:40:42 UTC) #8
danakj
LGTM
4 years, 11 months ago (2016-01-04 23:41:08 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1553263002/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1553263002/20001
4 years, 11 months ago (2016-01-04 23:42:13 UTC) #13
sandersd (OOO until July 31)
lgtm
4 years, 11 months ago (2016-01-04 23:45:26 UTC) #14
commit-bot: I haz the power
Try jobs failed on following builders: chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_chromium_compile_only_ng/builds/75135)
4 years, 11 months ago (2016-01-05 00:36:36 UTC) #16
dcheng
Annnnd TBRing spang@ for //ui/events/ozone/evdev changes too.
4 years, 11 months ago (2016-01-05 01:09:21 UTC) #20
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1553263002/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1553263002/40001
4 years, 11 months ago (2016-01-05 01:09:40 UTC) #22
commit-bot: I haz the power
Try jobs failed on following builders: linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_gn_chromeos_rel/builds/125245)
4 years, 11 months ago (2016-01-05 01:51:11 UTC) #24
dcheng
4 years, 11 months ago (2016-01-05 01:57:18 UTC) #26
dcheng
TBRing sky as well for //ash
4 years, 11 months ago (2016-01-05 01:57:25 UTC) #27
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1553263002/60001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1553263002/60001
4 years, 11 months ago (2016-01-05 01:57:58 UTC) #31
commit-bot: I haz the power
Try jobs failed on following builders: linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_chromeos_ozone_rel_ng/builds/106781)
4 years, 11 months ago (2016-01-05 02:35:33 UTC) #33
spang
lgtm
4 years, 11 months ago (2016-01-05 18:48:06 UTC) #35
spang
On 2016/01/05 18:48:06, spang wrote: > lgtm % build failures of course
4 years, 11 months ago (2016-01-05 18:48:19 UTC) #36
dcheng
OK, there are far too many feature-specific defines/flags on the CroS bots... Fixed //components/exo and ...
4 years, 11 months ago (2016-01-07 01:30:37 UTC) #41
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1553263002/100001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1553263002/100001
4 years, 11 months ago (2016-01-07 01:32:46 UTC) #43
commit-bot: I haz the power
Committed patchset #6 (id:100001)
4 years, 11 months ago (2016-01-07 03:55:03 UTC) #44
commit-bot: I haz the power
4 years, 11 months ago (2016-01-07 03:56:01 UTC) #46
Message was sent while issue was closed.
Patchset 6 (id:??) landed as
https://crrev.com/78820a435f2d94fb08e9a4cdb810b811c6cbdea9
Cr-Commit-Position: refs/heads/master@{#368009}

Powered by Google App Engine
This is Rietveld 408576698