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

Issue 71203003: Delete dbus/ibus directory. (Closed)

Created:
7 years, 1 month ago by Hiro Komatsu
Modified:
7 years, 1 month ago
Reviewers:
satorux1, Seigo Nonaka
CC:
chromium-reviews, stevenjb+watch_chromium.org, oshima+watch_chromium.org, Seigo Nonaka, Zachary Kuznia, Yusuke Sato
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Visibility:
Public.

Description

Delete dbus/ibus directory. This is the last patch to delete dbus/ibus directory. BUG=275262 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=235739

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -69 lines) Patch
M chromeos/chromeos.gyp View 1 chunk +0 lines, -1 line 0 comments Download
M chromeos/dbus/dbus_thread_manager.h View 1 chunk +0 lines, -2 lines 0 comments Download
D chromeos/dbus/ibus/OWNERS View 1 chunk +0 lines, -1 line 0 comments Download
D chromeos/dbus/ibus/ibus_constants.h View 1 chunk +0 lines, -65 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Hiro Komatsu
This patch depends on: * https://codereview.chromium.org/68503002/ * https://codereview.chromium.org/68603002/ * https://codereview.chromium.org/70053003/ * https://codereview.chromium.org/61003004/
7 years, 1 month ago (2013-11-13 11:45:39 UTC) #1
Seigo Nonaka
lgtm! awesome!!!
7 years, 1 month ago (2013-11-13 14:11:28 UTC) #2
satorux1
lgtm!
7 years, 1 month ago (2013-11-13 14:24:59 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/komatsu@chromium.org/71203003/1
7 years, 1 month ago (2013-11-18 13:38:20 UTC) #4
commit-bot: I haz the power
7 years, 1 month ago (2013-11-18 16:27:34 UTC) #5
Message was sent while issue was closed.
Change committed as 235739

Powered by Google App Engine
This is Rietveld 408576698