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

Issue 10900041: dbus: Fix clang warnings about missing virtual and OVERRIDE annotations. (Closed)

Created:
8 years, 3 months ago by tfarina
Modified:
8 years, 3 months ago
Reviewers:
satorux1
CC:
chromium-reviews
Visibility:
Public.

Description

dbus: Fix clang warnings about missing virtual and OVERRIDE annotations. BUG=115047 TBR=satorux@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=154116

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -3 lines) Patch
M dbus/bus.cc View 2 chunks +3 lines, -3 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
tfarina
TBR + CQ.
8 years, 3 months ago (2012-08-30 02:55:02 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tfarina@chromium.org/10900041/1
8 years, 3 months ago (2012-08-30 02:55:09 UTC) #2
satorux1
LGTM
8 years, 3 months ago (2012-08-30 04:29:40 UTC) #3
commit-bot: I haz the power
8 years, 3 months ago (2012-08-30 04:32:49 UTC) #4
Try job failure for 10900041-1 (retry) on win for step "compile" (clobber
build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win&number...

Powered by Google App Engine
This is Rietveld 408576698