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

Issue 2309973002: dbus: Remove dbus::FileDescriptor from MediaTransferProtocolDaemonClient (Closed)

Created:
4 years, 3 months ago by hashimoto
Modified:
4 years, 3 months ago
Reviewers:
satorux1
CC:
chromium-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

dbus: Remove dbus::FileDescriptor from MediaTransferProtocolDaemonClient BUG=621841 Committed: https://crrev.com/55cfa99f538429f875943ca54d3673a7394dc6e1 Cr-Commit-Position: refs/heads/master@{#416602}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -4 lines) Patch
M device/media_transfer_protocol/media_transfer_protocol_daemon_client.cc View 1 chunk +1 line, -4 lines 0 comments Download

Messages

Total messages: 15 (9 generated)
hashimoto
4 years, 3 months ago (2016-09-06 06:52:10 UTC) #2
satorux1
lgtm
4 years, 3 months ago (2016-09-06 07:25:42 UTC) #5
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/2309973002/1
4 years, 3 months ago (2016-09-06 07:30:18 UTC) #9
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/2309973002/1
4 years, 3 months ago (2016-09-06 07:33:04 UTC) #12
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 3 months ago (2016-09-06 07:34:12 UTC) #13
commit-bot: I haz the power
4 years, 3 months ago (2016-09-06 07:36:27 UTC) #15
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/55cfa99f538429f875943ca54d3673a7394dc6e1
Cr-Commit-Position: refs/heads/master@{#416602}

Powered by Google App Engine
This is Rietveld 408576698