Rename test callbacks for CookieStore to represent the argument type.

Rename test callbacks for CookieStore to represent the argument type.

These callbacks are not tightly connected to each method but are made to save
a result value of the specified type. Rename them to represent the type.

Actually, SetCookieCallback is used for testing Delete method which passes
a bool result to the done callback.

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

[erikwright (departed), 2013-05-22 01:33:33]

LGTM. Bonus points for coalescing the three redundant class definitions into a
template like CookieResultCallback<std::string>. You just need a tiny bit of
pixie dust to remove the const/ref from std::string when declaring the member
for storing the value.

[tyoshino (SeeGerritForStatus), 2013-05-22 02:26:05]

On 2013/05/22 01:33:33, erikwright wrote:
> LGTM. Bonus points for coalescing the three redundant class definitions into a
> template like CookieResultCallback<std::string>. You just need a tiny bit of
> pixie dust to remove the const/ref from std::string when declaring the member
> for storing the value.

Thanks for review. I was thinking on that too. Let me do that in the next CL.

[commit-bot: I haz the power, 2013-05-22 02:26:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tyoshino@chromium.org/15380003/1

[commit-bot: I haz the power, 2013-05-22 12:43:41]

Change committed as 201497