Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(922)

Issue 2583343002: Mark OnceCallback::Run() & const so it matches a const callback object. (Closed)

Created:
1 year, 4 months ago by dcheng
Modified:
1 year, 4 months ago
Reviewers:
vmpstr
CC:
chromium-reviews, vmpstr+watch_chromium.org, Avi (use Gerrit)
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Mark OnceCallback::Run() & const so it matches a const callback object. BUG=675328 Committed: https://crrev.com/3f2fd66ee3c869dbf43bf3c44877406ecfd871f3 Cr-Commit-Position: refs/heads/master@{#439938}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -4 lines) Patch
M base/bind_unittest.nc View 1 chunk +15 lines, -1 line 0 comments Download
M base/callback.h View 2 chunks +3 lines, -3 lines 0 comments Download

Messages

Total messages: 17 (12 generated)
dcheng
Before, it'd just complain that neither of the two Run() overloads matched: ../../base/bind_unittest.cc:1420:6: error: no ...
1 year, 4 months ago (2016-12-19 20:02:53 UTC) #4
vmpstr
lgtm, the error is much better! thanks
1 year, 4 months ago (2016-12-20 19:13:43 UTC) #8
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/2583343002/1
1 year, 4 months ago (2016-12-20 19:15:54 UTC) #12
commit-bot: I haz the power
Committed patchset #1 (id:1)
1 year, 4 months ago (2016-12-20 23:57:31 UTC) #15
commit-bot: I haz the power
1 year, 4 months ago (2016-12-21 00:00:15 UTC) #17
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/3f2fd66ee3c869dbf43bf3c44877406ecfd871f3
Cr-Commit-Position: refs/heads/master@{#439938}

Powered by Google App Engine
This is Rietveld 408576698