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

Issue 14985008: dbus: adding PostTaskToDBusThreadAndReply (Closed)

Created:
7 years, 7 months ago by Haruki Sato
Modified:
7 years, 7 months ago
Reviewers:
satorux1
CC:
chromium-reviews
Visibility:
Public.

Description

dbus: adding PostTaskToDBusThreadAndReply This CL only adds a util method. BUG=172821 TEST=unittests R=satorux@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=198876

Patch Set 1 #

Patch Set 2 : remove a blank line. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -0 lines) Patch
M dbus/bus.h View 1 chunk +7 lines, -0 lines 0 comments Download
M dbus/bus.cc View 1 1 chunk +18 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Haruki Sato
7 years, 7 months ago (2013-05-08 06:06:16 UTC) #1
satorux1
LGTM. Could you clean up the existing code using this?
7 years, 7 months ago (2013-05-08 06:11:48 UTC) #2
Haruki Sato
On 2013/05/08 06:11:48, satorux1 wrote: > LGTM. Could you clean up the existing code using ...
7 years, 7 months ago (2013-05-08 11:56:07 UTC) #3
Haruki Sato
7 years, 7 months ago (2013-05-08 11:57:26 UTC) #4
Message was sent while issue was closed.
Committed patchset #2 manually as r198876 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698