Revert of Convert network hints to Mojo

Revert of Convert network hints to Mojo (patchset #16 id:300001 of https://codereview.chromium.org/2144533002/ )

BUG=630749
NOPRESUBMIT=true

Skipping presubmit as I'm removing OWNERS files I added.

Reason for revert:
Use-after-free issue:

https://bugs.chromium.org/p/chromium/issues/detail?id=630749

Original issue's description:
> Convert network hints to Mojo
>
> Previously landed as: refs/heads/master@{#406780}
>
> Committed: https://crrev.com/1efd0ad3153587609220eeec1be6c1b2bbf9964b
> Cr-Commit-Position: refs/heads/master@{#407030}

TBR=sammc@chromium.org,dcheng@chromium.org,juliatuttle@chromium.org,halliwell@chromium.org,sky@chromium.org,mbarbella@chromium.org,blundell@chromium.org,mbarbella@google.com
# Not skipping CQ checks because original CL landed more than 1 days ago.

Committed: https://crrev.com/0773f761ce9ed103aa297808ac89771bc010d70c
Cr-Commit-Position: refs/heads/master@{#407395}

[tibell, 2016-07-24 23:51:59]

The CQ bit was checked by tibell@chromium.org

[tibell, 2016-07-24 23:52:00]

Created Revert of Convert network hints to Mojo

[commit-bot: I haz the power, 2016-07-24 23:52:03]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-24 23:52:04]

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

[commit-bot: I haz the power, 2016-07-24 23:52:05]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Sam McNally, 2016-07-24 23:52:45]

The CQ bit was checked by sammc@chromium.org

[Sam McNally, 2016-07-24 23:52:45]

lgtm

[tibell, 2016-07-24 23:52:46]

Description was changed from

==========
Revert of Convert network hints to Mojo (patchset #16 id:300001 of
https://codereview.chromium.org/2144533002/ )

Reason for revert:
Use-after-free issue:

https://bugs.chromium.org/p/chromium/issues/detail?id=630749

Original issue's description:
> Convert network hints to Mojo
> 
> Previously landed as: refs/heads/master@{#406780}
> 
> Committed: https://crrev.com/1efd0ad3153587609220eeec1be6c1b2bbf9964b
> Cr-Commit-Position: refs/heads/master@{#407030}

TBR=sammc@chromium.org,dcheng@chromium.org,juliatuttle@chromium.org,halliwell...
# Not skipping CQ checks because original CL landed more than 1 days ago.
==========

to

==========
Revert of Convert network hints to Mojo (patchset #16 id:300001 of
https://codereview.chromium.org/2144533002/ )

BUG=630749

Reason for revert:
Use-after-free issue:

https://bugs.chromium.org/p/chromium/issues/detail?id=630749

Original issue's description:
> Convert network hints to Mojo
> 
> Previously landed as: refs/heads/master@{#406780}
> 
> Committed: https://crrev.com/1efd0ad3153587609220eeec1be6c1b2bbf9964b
> Cr-Commit-Position: refs/heads/master@{#407030}

TBR=sammc@chromium.org,dcheng@chromium.org,juliatuttle@chromium.org,halliwell...
# Not skipping CQ checks because original CL landed more than 1 days ago.
==========

[commit-bot: I haz the power, 2016-07-24 23:52:53]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-24 23:58:06]

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

[commit-bot: I haz the power, 2016-07-24 23:58:08]

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

[tibell, 2016-07-25 00:09:30]

Description was changed from

==========
Revert of Convert network hints to Mojo (patchset #16 id:300001 of
https://codereview.chromium.org/2144533002/ )

BUG=630749

Reason for revert:
Use-after-free issue:

https://bugs.chromium.org/p/chromium/issues/detail?id=630749

Original issue's description:
> Convert network hints to Mojo
> 
> Previously landed as: refs/heads/master@{#406780}
> 
> Committed: https://crrev.com/1efd0ad3153587609220eeec1be6c1b2bbf9964b
> Cr-Commit-Position: refs/heads/master@{#407030}

TBR=sammc@chromium.org,dcheng@chromium.org,juliatuttle@chromium.org,halliwell...
# Not skipping CQ checks because original CL landed more than 1 days ago.
==========

to

==========
Revert of Convert network hints to Mojo (patchset #16 id:300001 of
https://codereview.chromium.org/2144533002/ )

BUG=630749
NOPRESUBMIT=true

Skipping presubmit as I'm removing OWNERS files I added.

Reason for revert:
Use-after-free issue:

https://bugs.chromium.org/p/chromium/issues/detail?id=630749

Original issue's description:
> Convert network hints to Mojo
> 
> Previously landed as: refs/heads/master@{#406780}
> 
> Committed: https://crrev.com/1efd0ad3153587609220eeec1be6c1b2bbf9964b
> Cr-Commit-Position: refs/heads/master@{#407030}

TBR=sammc@chromium.org,dcheng@chromium.org,juliatuttle@chromium.org,halliwell...
# Not skipping CQ checks because original CL landed more than 1 days ago.
==========

[tibell, 2016-07-25 00:09:43]

The CQ bit was checked by tibell@chromium.org

[commit-bot: I haz the power, 2016-07-25 00:09:52]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-25 01:02:24]

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

[commit-bot: I haz the power, 2016-07-25 01:02:26]

Failed to apply patch for
components/network_hints/browser/network_hints_impl.cc:
While running git rm components/network_hints/browser/network_hints_impl.cc;
  fatal: pathspec 'components/network_hints/browser/network_hints_impl.cc' did
not match any files

Patch:  D    components/network_hints/browser/network_hints_impl.cc
Index: components/network_hints/browser/network_hints_impl.cc
diff --git a/components/network_hints/browser/network_hints_impl.cc
b/components/network_hints/browser/network_hints_impl.cc
deleted file mode 100644
index
1462d58d5be82de249289fe36ded0e87248df4e8..0000000000000000000000000000000000000000
--- a/components/network_hints/browser/network_hints_impl.cc
+++ /dev/null
@@ -1,86 +0,0 @@
-// Copyright 2014 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "components/network_hints/browser/network_hints_impl.h"
-
-#include "base/logging.h"
-#include "base/macros.h"
-#include "components/network_hints/common/network_hints_common.h"
-#include "ipc/ipc_message_macros.h"
-#include "net/base/address_list.h"
-#include "net/base/net_errors.h"
-#include "net/dns/host_resolver.h"
-#include "net/dns/single_request_host_resolver.h"
-#include "url/gurl.h"
-
-namespace network_hints {
-
-namespace {
-
-const int kDefaultPort = 80;
-
-class DnsLookupRequest {
- public:
-  DnsLookupRequest(net::HostResolver* host_resolver,
-                   const std::string& hostname)
-      : hostname_(hostname),
-        resolver_(host_resolver) {
-  }
-
-  // Return underlying network resolver status.
-  // net::OK ==> Host was found synchronously.
-  // net:ERR_IO_PENDING ==> Network will callback later with result.
-  // anything else ==> Host was not found synchronously.
-  int Start() {
-    net::HostResolver::RequestInfo resolve_info(
-        net::HostPortPair(hostname_, kDefaultPort));
-
-    // Make a note that this is a speculative resolve request. This allows
-    // separating it from real navigations in the observer's callback, and
-    // lets the HostResolver know it can be de-prioritized.
-    resolve_info.set_is_speculative(true);
-    return resolver_.Resolve(
-        resolve_info,
-        net::DEFAULT_PRIORITY,
-        &addresses_,
-        base::Bind(&DnsLookupRequest::OnLookupFinished, base::Owned(this)),
-        net::BoundNetLog());
-  }
-
- private:
-  void OnLookupFinished(int result) {
-    VLOG(2) << __FUNCTION__ << ": " << hostname_ << ", result=" << result;
-  }
-
-  const std::string hostname_;
-  net::SingleRequestHostResolver resolver_;
-  net::AddressList addresses_;
-
-  DISALLOW_COPY_AND_ASSIGN(DnsLookupRequest);
-};
-
-}  // namespace
-
-void NetworkHintsImpl::Bind(mojom::NetworkHintsRequest request) {
-  bindings_.AddBinding(this, std::move(request));
-}
-
-NetworkHintsImpl::NetworkHintsImpl(net::HostResolver* host_resolver)
-    : host_resolver_(host_resolver) {
-  DCHECK(host_resolver_);
-}
-
-NetworkHintsImpl::~NetworkHintsImpl() {}
-
-void NetworkHintsImpl::DNSPrefetch(const LookupRequest& lookup_request) {
-  DCHECK(host_resolver_);
-  for (const std::string& hostname : lookup_request.hostname_list) {
-    DnsLookupRequest* request = new DnsLookupRequest(host_resolver_, hostname);
-    // Note: DnsLookupRequest will be freed by the base::Owned call when
-    // resolving has completed.
-    request->Start();
-  }
-}
-
-}  // namespace network_hints

[commit-bot: I haz the power, 2016-07-25 01:03:59]

Description was changed from

==========
Revert of Convert network hints to Mojo (patchset #16 id:300001 of
https://codereview.chromium.org/2144533002/ )

BUG=630749
NOPRESUBMIT=true

Skipping presubmit as I'm removing OWNERS files I added.

Reason for revert:
Use-after-free issue:

https://bugs.chromium.org/p/chromium/issues/detail?id=630749

Original issue's description:
> Convert network hints to Mojo
> 
> Previously landed as: refs/heads/master@{#406780}
> 
> Committed: https://crrev.com/1efd0ad3153587609220eeec1be6c1b2bbf9964b
> Cr-Commit-Position: refs/heads/master@{#407030}

TBR=sammc@chromium.org,dcheng@chromium.org,juliatuttle@chromium.org,halliwell...
# Not skipping CQ checks because original CL landed more than 1 days ago.
==========

to

==========
Revert of Convert network hints to Mojo (patchset #16 id:300001 of
https://codereview.chromium.org/2144533002/ )

BUG=630749
NOPRESUBMIT=true

Skipping presubmit as I'm removing OWNERS files I added.

Reason for revert:
Use-after-free issue:

https://bugs.chromium.org/p/chromium/issues/detail?id=630749

Original issue's description:
> Convert network hints to Mojo
>
> Previously landed as: refs/heads/master@{#406780}
>
> Committed: https://crrev.com/1efd0ad3153587609220eeec1be6c1b2bbf9964b
> Cr-Commit-Position: refs/heads/master@{#407030}

TBR=sammc@chromium.org,dcheng@chromium.org,juliatuttle@chromium.org,halliwell...
# Not skipping CQ checks because original CL landed more than 1 days ago.

Committed: https://crrev.com/0773f761ce9ed103aa297808ac89771bc010d70c
Cr-Commit-Position: refs/heads/master@{#407395}
==========

[commit-bot: I haz the power, 2016-07-25 01:04:01]

Patchset 1 (id:??) landed as
https://crrev.com/0773f761ce9ed103aa297808ac89771bc010d70c
Cr-Commit-Position: refs/heads/master@{#407395}