PrivetV2 SetupFlow, Session and SetupOperation interfaces

PrivetV2 SetupFlow, Session and SetupOperation interfaces

This is a merged CL containing the PrivetV2SetupFlow, PrivetV2Session,
and PrivetV2SetupOperation interfaces.

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

[Vitaly Buka (NO REVIEWS), 2014-05-23 17:17:10]

lgtm

BUG=372843

[commit-bot: I haz the power, 2014-05-23 17:17:37]

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

[Vitaly Buka (NO REVIEWS), 2014-05-23 17:22:09]

https://codereview.chromium.org/299123003/diff/1/chrome/browser/local_discove...
File chrome/browser/local_discovery/privetv2_session.h (right):

https://codereview.chromium.org/299123003/diff/1/chrome/browser/local_discove...
chrome/browser/local_discovery/privetv2_session.h:31: const std::string&
confirmation_code,
callbacks should be in Delegate scope

https://codereview.chromium.org/299123003/diff/1/chrome/browser/local_discove...
File chrome/browser/local_discovery/privetv2_setup_flow.h (right):

https://codereview.chromium.org/299123003/diff/1/chrome/browser/local_discove...
chrome/browser/local_discovery/privetv2_setup_flow.h:25: typedef
base::Callback<void(bool confirm)> ConfirmationCallback;
callback should be part of Delegate

[Noam Samuel, 2014-05-23 17:30:09]

https://codereview.chromium.org/299123003/diff/1/chrome/browser/local_discove...
File chrome/browser/local_discovery/privetv2_session.h (right):

https://codereview.chromium.org/299123003/diff/1/chrome/browser/local_discove...
chrome/browser/local_discovery/privetv2_session.h:31: const std::string&
confirmation_code,
On 2014/05/23 17:22:09, Vitaly Buka wrote:
> callbacks should be in Delegate scope

Done.

https://codereview.chromium.org/299123003/diff/1/chrome/browser/local_discove...
File chrome/browser/local_discovery/privetv2_setup_flow.h (right):

https://codereview.chromium.org/299123003/diff/1/chrome/browser/local_discove...
chrome/browser/local_discovery/privetv2_setup_flow.h:25: typedef
base::Callback<void(bool confirm)> ConfirmationCallback;
On 2014/05/23 17:22:09, Vitaly Buka wrote:
> callback should be part of Delegate

Done.

[Vitaly Buka (NO REVIEWS), 2014-05-23 17:32:28]

lgtm

[Noam Samuel, 2014-05-23 19:42:35]

On 2014/05/23 17:32:28, Vitaly Buka wrote:
> lgtm

Added changes from https://codereview.chromium.org/284323005/

[Noam Samuel, 2014-05-23 19:42:46]

The CQ bit was checked by noamsml@chromium.org

[commit-bot: I haz the power, 2014-05-23 19:43:56]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/noamsml@chromium.org/299123003/40001

[commit-bot: I haz the power, 2014-05-23 22:52:53]

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:
  android_dbg_triggered_tests on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/android_dbg_triggered...)

[commit-bot: I haz the power, 2014-05-24 00:07:21]

Change committed as 272647