Implement WebPrescientNetworking::prefetchDNS.

Implement WebPrescientNetworking::prefetchDNS.

This is to obsolete prefetchDNS() API which lives in platform API. I would like to move it under WebPrescientNetworking.

The corresponding Blink patch is http://crrev.com/15699002

BUG=None

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

[kouhei (in TOK), 2013-05-27 07:44:02]

jam: Would you take a look?

The corresponding Blink patch is http://crrev.com/15699002

Thanks!

[jam, 2013-05-28 16:07:03]

On 2013/05/27 07:44:02, kouhei wrote:
> jam: Would you take a look?
> 
> The corresponding Blink patch is http://crrev.com/15699002
> 
> Thanks!

see comment in chrome/OWNERS

[kouhei (in TOK), 2013-05-30 00:20:16]

thakis: Would you review this patch?

The corresponding Blink patch is http://crrev.com/15699002
Thanks!

[Nico, 2013-05-30 00:44:56]

Related patches should probably be mentioned in the CL description, not in a
comment.

I'm not very familiar with the preconnect code. I think jar@ might be, so I'm
trying to add him as an OWNER for this code in
https://codereview.chromium.org/15809005/ . Let's see how that goes :-)

https://codereview.chromium.org/15963006/diff/2001/chrome/renderer/net/presci...
File chrome/renderer/net/prescient_networking_dispatcher.cc (right):

https://codereview.chromium.org/15963006/diff/2001/chrome/renderer/net/presci...
chrome/renderer/net/prescient_networking_dispatcher.cc:9: #include
"content/public/common/content_client.h"
What do you need this for?

[kouhei (in TOK), 2013-05-30 00:54:08]

Thank you for your comments!

> Related patches should probably be mentioned in the CL description, not in a
comment.

Done.

https://codereview.chromium.org/15963006/diff/2001/chrome/renderer/net/presci...
File chrome/renderer/net/prescient_networking_dispatcher.cc (right):

https://codereview.chromium.org/15963006/diff/2001/chrome/renderer/net/presci...
chrome/renderer/net/prescient_networking_dispatcher.cc:9: #include
"content/public/common/content_client.h"
On 2013/05/30 00:44:57, Nico wrote:
> What do you need this for?
We don't need this.

Thank you for pointing this out. This was left over from old version of the
patch.

[Nico, 2013-05-30 01:10:04]

jar@ should probably look at this. lgtm once this lgthim.

[jar (doing other things), 2013-05-30 21:55:03]

Note that the DNS pre-resolution code that you're using tends to throttle the
rate at which resolutions can be requested.

  This tends to be helpful with scanning of pages (with a *ton* of names... such
as wikipedia).  I wanted to be sure that you were fine with that implementation
(which is really what you are calling into).  If you wanted more... we should
talk.

https://codereview.chromium.org/15963006/diff/8001/chrome/renderer/net/presci...
File chrome/renderer/net/prescient_networking_dispatcher.h (right):

https://codereview.chromium.org/15963006/diff/8001/chrome/renderer/net/presci...
chrome/renderer/net/prescient_networking_dispatcher.h:22: };
nit: Do we DISALLOW_COPY_AND_ASSIGN?  ...or is this living by webkit rules??

[kouhei (in TOK), 2013-05-31 05:56:29]

Thanks for your comments!

> Note that the DNS pre-resolution code that you're using tends to throttle the
rate at which resolutions can be requested.

Yes. I am aware of this.

https://codereview.chromium.org/15963006/diff/8001/chrome/renderer/net/presci...
File chrome/renderer/net/prescient_networking_dispatcher.h (right):

https://codereview.chromium.org/15963006/diff/8001/chrome/renderer/net/presci...
chrome/renderer/net/prescient_networking_dispatcher.h:22: };
On 2013/05/30 21:55:03, jar wrote:
> nit: Do we DISALLOW_COPY_AND_ASSIGN?  ...or is this living by webkit rules??

Done.

[jar (doing other things), 2013-05-31 17:23:13]

lgtm

[commit-bot: I haz the power, 2013-06-05 00:23:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kouhei@chromium.org/15963006/14001

[commit-bot: I haz the power, 2013-06-05 00:56:25]

Sorry for I got bad news for ya.
Compile failed with a clobber build on linux_chromeos_clang.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-06-07 16:44:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kouhei@chromium.org/15963006/33001

[commit-bot: I haz the power, 2013-06-07 17:06:21]

Sorry for I got bad news for ya.
Compile failed with a clobber build on linux_chromeos_clang.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-06-07 18:21:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kouhei@chromium.org/15963006/58001

[commit-bot: I haz the power, 2013-06-07 21:39:13]

Change committed as 204938