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

Issue 8846003: Make callback.h include callback_forward.h and add comments about forward declaration. (Closed)

Created:
9 years ago by erikwright (departed)
Modified:
9 years ago
CC:
chromium-reviews, brettw-cc_chromium.org, scheib
Visibility:
Public.

Description

Make callback.h include callback_forward.h and add comments about forward declaration. The first bit is designed to ensure that the correctness of callback_forward.h is tested by the callback unittests. The second bit is designed to increase the likelihood of developers actually using callback_forward.h as intended. BUG=None TEST=None Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=113472

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -0 lines) Patch
M base/callback.h View 2 chunks +7 lines, -0 lines 0 comments Download
M base/callback.h.pump View 2 chunks +7 lines, -0 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
erikwright (departed)
Hi William, PTAL. I'm very open to moving the comments about or adjusting the language ...
9 years ago (2011-12-07 16:24:10 UTC) #1
willchan no longer on Chromium
LGTM, but please wait for ajwong's review.
9 years ago (2011-12-07 16:25:34 UTC) #2
erikwright (departed)
On 2011/12/07 16:25:34, willchan wrote: > LGTM, but please wait for ajwong's review. Sorry, I ...
9 years ago (2011-12-07 16:26:35 UTC) #3
willchan no longer on Chromium
Hehe, ajwong is definitely the callback guy. I'm just the rubberstamper and receiver of painful ...
9 years ago (2011-12-07 16:30:34 UTC) #4
awong
LGTM @willchan: there's a correlation between good reviewers and getting hard reviews. :)
9 years ago (2011-12-07 18:41:20 UTC) #5
awong
oh, please fix up the BUG and TEST lines before committing. On 2011/12/07 18:41:20, awong ...
9 years ago (2011-12-07 18:41:56 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/erikwright@chromium.org/8846003/1
9 years ago (2011-12-07 18:51:39 UTC) #7
commit-bot: I haz the power
9 years ago (2011-12-07 21:44:07 UTC) #8
Change committed as 113472

Powered by Google App Engine
This is Rietveld 408576698