Remove ServiceDiscoveryClient* from PrivetHTTPAsynchronousFactoryImpl interface

Remove ServiceDiscoveryClient* from PrivetHTTPAsynchronousFactoryImpl interface

PrivetHTTPAsynchronousFactoryImpl can use ServiceDiscoverySharedClient::GetInstance.
Also we don't use this for testing, so we can simplify interface.

BUG=461504
Committed: https://crrev.com/e61a6220ca4341f7eafac6dc3aacaa6f80a89896
Cr-Commit-Position: refs/heads/master@{#318013}

[Vitaly Buka (NO REVIEWS), 2015-02-25 01:13:14]

vitalybuka@chromium.org changed reviewers:
+ alekseys@chromium.org

[Aleksey Shlyapnikov, 2015-02-25 01:35:38]

lgtm

https://codereview.chromium.org/959503003/diff/60001/chrome/browser/local_dis...
File chrome/browser/local_discovery/privet_http_asynchronous_factory.h (right):

https://codereview.chromium.org/959503003/diff/60001/chrome/browser/local_dis...
chrome/browser/local_discovery/privet_http_asynchronous_factory.h:21: class
ServiceDiscoverySharedClient;
Why do you need this declaration in this header file?

[Vitaly Buka (NO REVIEWS), 2015-02-25 01:37:09]

https://codereview.chromium.org/959503003/diff/60001/chrome/browser/local_dis...
File chrome/browser/local_discovery/privet_http_asynchronous_factory.h (right):

https://codereview.chromium.org/959503003/diff/60001/chrome/browser/local_dis...
chrome/browser/local_discovery/privet_http_asynchronous_factory.h:21: class
ServiceDiscoverySharedClient;
On 2015/02/25 01:35:38, Aleksey Shlyapnikov wrote:
> Why do you need this declaration in this header file?

thanks, I don't

[Vitaly Buka (NO REVIEWS), 2015-02-25 01:40:41]

The CQ bit was checked by vitalybuka@chromium.org

[Vitaly Buka (NO REVIEWS), 2015-02-25 01:40:41]

The patchset sent to the CQ was uploaded after l-g-t-m from
alekseys@chromium.org
Link to the patchset: https://codereview.chromium.org/959503003/#ps80001 (title:
"Tue Feb 24 17:40:14 PST 2015")

[commit-bot: I haz the power, 2015-02-25 01:41:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/959503003/80001

[commit-bot: I haz the power, 2015-02-25 05:35:00]

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

[commit-bot: I haz the power, 2015-02-25 05:35:01]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Vitaly Buka (NO REVIEWS), 2015-02-25 07:12:19]

The CQ bit was checked by vitalybuka@chromium.org

[Vitaly Buka (NO REVIEWS), 2015-02-25 07:12:20]

The patchset sent to the CQ was uploaded after l-g-t-m from
alekseys@chromium.org
Link to the patchset: https://codereview.chromium.org/959503003/#ps100001
(title: "Tue Feb 24 23:11:53 PST 2015")

[commit-bot: I haz the power, 2015-02-25 07:13:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/959503003/100001

[commit-bot: I haz the power, 2015-02-25 09:47:22]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2015-02-25 09:48:06]

Patchset 6 (id:??) landed as
https://crrev.com/e61a6220ca4341f7eafac6dc3aacaa6f80a89896
Cr-Commit-Position: refs/heads/master@{#318013}