Make the simple QUIC client able to resolve hosts.

Make the simple QUIC client able to resolve hosts.
Implement a SynchronousHostResolver for this purpose.

Committed: https://crrev.com/acfb2f91085d5291ba1c9a04dbe7aea610b13e33
Cr-Commit-Position: refs/heads/master@{#321688}

Committed: https://crrev.com/da78df5aee4e4ebdf62b5b2bc17056ce5db24b61
Cr-Commit-Position: refs/heads/master@{#321708}

[Ryan Hamilton, 2015-03-20 17:29:02]

rch@chromium.org changed reviewers:
+ rtenneti@chromium.org

[ramant (doing other things), 2015-03-20 17:46:56]

Thanks very much for doing this.

LGTM.

https://codereview.chromium.org/1004173004/diff/40001/net/tools/quic/synchron...
File net/tools/quic/synchronous_host_resolver.cc (right):

https://codereview.chromium.org/1004173004/diff/40001/net/tools/quic/synchron...
net/tools/quic/synchronous_host_resolver.cc:1: #include
"net/tools/quic/synchronous_host_resolver.h"
nit: copyright

https://codereview.chromium.org/1004173004/diff/40001/net/tools/quic/synchron...
File net/tools/quic/synchronous_host_resolver.h (right):

https://codereview.chromium.org/1004173004/diff/40001/net/tools/quic/synchron...
net/tools/quic/synchronous_host_resolver.h:5: // A toy client, which connects to
a specified port and sends QUIC
nit: Consider either updating/deleting the comment.

[Ryan Hamilton, 2015-03-20 21:38:48]

Thanks!

https://codereview.chromium.org/1004173004/diff/40001/net/tools/quic/synchron...
File net/tools/quic/synchronous_host_resolver.cc (right):

https://codereview.chromium.org/1004173004/diff/40001/net/tools/quic/synchron...
net/tools/quic/synchronous_host_resolver.cc:1: #include
"net/tools/quic/synchronous_host_resolver.h"
On 2015/03/20 17:46:56, ramant wrote:
> nit: copyright

Done.

https://codereview.chromium.org/1004173004/diff/40001/net/tools/quic/synchron...
File net/tools/quic/synchronous_host_resolver.h (right):

https://codereview.chromium.org/1004173004/diff/40001/net/tools/quic/synchron...
net/tools/quic/synchronous_host_resolver.h:5: // A toy client, which connects to
a specified port and sends QUIC
On 2015/03/20 17:46:56, ramant wrote:
> nit: Consider either updating/deleting the comment.

Good point! Done.

[Ryan Hamilton, 2015-03-20 21:38:52]

The CQ bit was checked by rch@chromium.org

[Ryan Hamilton, 2015-03-20 21:38:52]

The patchset sent to the CQ was uploaded after l-g-t-m from
rtenneti@chromium.org
Link to the patchset: https://codereview.chromium.org/1004173004/#ps80001
(title: "Rebase")

[commit-bot: I haz the power, 2015-03-20 21:39:37]

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

[Ryan Hamilton, 2015-03-20 21:40:25]

The CQ bit was unchecked by rch@chromium.org

[Ryan Hamilton, 2015-03-20 21:42:41]

The CQ bit was checked by rch@chromium.org

[Ryan Hamilton, 2015-03-20 21:42:42]

The patchset sent to the CQ was uploaded after l-g-t-m from
rtenneti@chromium.org
Link to the patchset: https://codereview.chromium.org/1004173004/#ps100001
(title: "GN")

[commit-bot: I haz the power, 2015-03-20 21:43:49]

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

[commit-bot: I haz the power, 2015-03-20 22:00:29]

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

[commit-bot: I haz the power, 2015-03-20 22:00:30]

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

[Ryan Hamilton, 2015-03-20 22:42:04]

The CQ bit was checked by rch@chromium.org

[Ryan Hamilton, 2015-03-20 22:42:04]

The patchset sent to the CQ was uploaded after l-g-t-m from
rtenneti@chromium.org
Link to the patchset: https://codereview.chromium.org/1004173004/#ps120001
(title: "Add simple_quic_tools as a dep of quic_client")

[commit-bot: I haz the power, 2015-03-20 22:42:53]

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

[commit-bot: I haz the power, 2015-03-21 00:39:07]

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

[commit-bot: I haz the power, 2015-03-21 00:39:08]

Try jobs failed on following builders:
  win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[Ryan Hamilton, 2015-03-21 01:46:10]

The CQ bit was checked by rch@chromium.org

[commit-bot: I haz the power, 2015-03-21 01:47:14]

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

[commit-bot: I haz the power, 2015-03-21 07:42:16]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2015-03-21 07:42:53]

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

[pfeldman, 2015-03-21 13:15:13]

A revert of this CL (patchset #7 id:120001) has been created in
https://codereview.chromium.org/1025133002/ by pfeldman@chromium.org.

The reason for reverting is: Build failure:
http://build.chromium.org/p/chromium.win/builders/Win%20x64%20Builder%20%28db....

[Ryan Hamilton, 2015-03-21 14:12:54]

The CQ bit was checked by rch@chromium.org

[Ryan Hamilton, 2015-03-21 14:12:55]

The patchset sent to the CQ was uploaded after l-g-t-m from
rtenneti@chromium.org
Link to the patchset: https://codereview.chromium.org/1004173004/#ps140001
(title: "Remove EXPORT to fix Win")

[commit-bot: I haz the power, 2015-03-21 14:13:01]

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

[commit-bot: I haz the power, 2015-03-22 05:16:47]

Committed patchset #8 (id:140001)

[commit-bot: I haz the power, 2015-03-22 05:18:17]

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