Move AcceptWithResponder() out of the MessageReceiver interface.

Move AcceptWithResponder() out of the MessageReceiver interface.

BUG=None
TEST=None

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=272831

[yzshen1, 2014-05-22 19:58:23]

Hi, Darin.

Would you please take a look?

I think this results in more clear interface, but please feel free to reject. :)

Thanks!

[darin (slow to review), 2014-05-22 23:55:39]

This is a great patch! We just need a better name ;-)

[yzshen1, 2014-05-22 23:56:55]

On 2014/05/22 23:55:39, darin wrote:
> This is a great patch! We just need a better name ;-)

Thanks, I will think harder and get back to you soon! :)

[qsr, 2014-05-23 09:10:09]

I was thinking about doing something like this on the java implementation. Good
to know it will be the same in C++.

[yzshen1, 2014-05-23 16:34:49]

Darin: Would you please take another look? Thanks!

On 2014/05/23 09:10:09, qsr wrote:
> I was thinking about doing something like this on the java implementation.
Good
> to know it will be the same in C++.

Cool. :)

[darin (slow to review), 2014-05-24 04:15:07]

LGTM

[yzshen1, 2014-05-24 04:37:50]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2014-05-24 04:38:08]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/yzshen@chromium.org/299833008/10001

[commit-bot: I haz the power, 2014-05-24 06:26:12]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[commit-bot: I haz the power, 2014-05-24 06:31:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-05-24 06:31:22]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[yzshen1, 2014-05-26 07:11:48]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2014-05-26 07:12:38]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/yzshen@chromium.org/299833008/30001

[commit-bot: I haz the power, 2014-05-26 15:57:10]

Change committed as 272831