wifi_sync: add FakeWifiConfigDelegate

wifi_sync: add FakeWifiConfigDelegate

FakeWifiConfigDelegate is a simple fake, which gives us the ability
to tell how many times a method is has been called.

BUG=chromium:431435
TEST=components_unittests --gtest_filter="Wifi*"

[mukesh agrawal, 2015-01-12 20:53:24]

quiche@chromium.org changed reviewers:
+ erikwright@chromium.org, stevenjb@chromium.org

[mukesh agrawal, 2015-01-12 20:53:24]

quiche@chromium.org changed required reviewers:
+ erikwright@chromium.org, stevenjb@chromium.org

[mukesh agrawal, 2015-01-12 20:53:25]

Seems to be uncommon to write unit tests for fakes, so I've omitted unit tests
for this CL.

[mukesh agrawal, 2015-01-12 20:54:31]

quiche@chromium.org changed reviewers:
+ pstew@chromium.org

[erikwright (departed), 2015-01-12 20:58:09]

Do you already have multiple unit-tests that will use this?

It's helpful to include links to dependent CLs if you have them.

[mukesh agrawal, 2015-01-12 21:05:16]

On 2015/01/12 20:58:09, erikwright wrote:
> Do you already have multiple unit-tests that will use this?
> 
> It's helpful to include links to dependent CLs if you have them.

Sure thing. This fake will be used by WifiCredentialSyncableService unit tests,
in https://codereview.chromium.org/836303004/.

The fake is used in ProcessSyncChangesNotStarted, and
ProcessSyncChangesInvalidChange.

[erikwright (departed), 2015-01-13 18:39:08]

On 2015/01/12 21:05:16, mukesh agrawal wrote:
> On 2015/01/12 20:58:09, erikwright wrote:
> > Do you already have multiple unit-tests that will use this?
> > 
> > It's helpful to include links to dependent CLs if you have them.
> 
> Sure thing. This fake will be used by WifiCredentialSyncableService unit
tests,
> in https://codereview.chromium.org/836303004/.
> 
> The fake is used in ProcessSyncChangesNotStarted, and
> ProcessSyncChangesInvalidChange.

If it's only used by a single test declare it directly in the unittest.cc file.

[stevenjb, 2015-01-13 23:02:01]

On 2015/01/13 18:39:08, erikwright wrote:
> On 2015/01/12 21:05:16, mukesh agrawal wrote:
> > On 2015/01/12 20:58:09, erikwright wrote:
> > > Do you already have multiple unit-tests that will use this?
> > > 
> > > It's helpful to include links to dependent CLs if you have them.
> > 
> > Sure thing. This fake will be used by WifiCredentialSyncableService unit
> tests,
> > in https://codereview.chromium.org/836303004/.
> > 
> > The fake is used in ProcessSyncChangesNotStarted, and
> > ProcessSyncChangesInvalidChange.
> 
> If it's only used by a single test declare it directly in the unittest.cc
file.

+1

If we end up using it in more than one place, we can pull it out to a separate
file at that point.

[mukesh agrawal, 2015-01-14 01:16:50]

Rolling into next CL. (Will move the code into the WCSS unit tests file.)