Client config and QUIC transaction integration tests

Client config and QUIC transaction integration tests

BUG=680570, 680564
Review-Url: https://codereview.chromium.org/2754803004
Cr-Commit-Position: refs/heads/master@{#457892}
Committed: https://chromium.googlesource.com/chromium/src/+/e9ee3d63b09c9ef3f9dafc84bf7cfa83c89203fc

[tbansal1, 2017-03-17 00:09:56]

Patchset #1 (id:1) has been deleted

[tbansal1, 2017-03-17 00:10:29]

Description was changed from

==========
Client config integration test

BUG=
==========

to

==========
Client config and QUIC transaction integration tests

BUG=680570, 680564
==========

[tbansal1, 2017-03-17 00:13:10]

tbansal@chromium.org changed reviewers:
+ ryansturm@chromium.org

[tbansal1, 2017-03-17 00:13:11]

ryansturm: ptal. Thanks.

[tbansal1, 2017-03-17 17:17:13]

tbansal@chromium.org changed reviewers:
+ megjablon@chromium.org
- ryansturm@chromium.org

[tbansal1, 2017-03-17 17:17:24]

megjablon: ptal. Thanks.

[tbansal1, 2017-03-17 18:14:56]

tbansal@chromium.org changed reviewers:
+ robertogden@chromium.org

[tbansal1, 2017-03-17 18:14:56]

ptal. Thanks.

[Robert Ogden, 2017-03-17 18:35:40]

lgtm

[megjablon, 2017-03-17 21:38:07]

The bugs say these should be in client_config.py and quic.py. Any reason they're
in smoke instead?

[Robert Ogden, 2017-03-17 21:42:03]

On 2017/03/17 21:38:07, megjablon wrote:
> The bugs say these should be in client_config.py and quic.py. Any reason
they're
> in smoke instead?

The file names were just made up by me. But I defer to whatever you think is
cleanest

[megjablon, 2017-03-17 21:45:51]

lgtm

[tbansal1, 2017-03-17 22:01:51]

On 2017/03/17 21:45:51, megjablon wrote:
> lgtm

I do not have any preference on the filename. I did not
see that the filename was specified in the bug.

I think once we have some experience with buildbots,
we can probably rearrange?
Both client config and quic have only one test. 
So, for now it is probably OK to keep them there instead
of creating a separate file.

[tbansal1, 2017-03-17 22:01:58]

The CQ bit was checked by tbansal@chromium.org

[commit-bot: I haz the power, 2017-03-17 22:04:15]

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, 2017-03-17 22:21:42]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489788118433600,
"parent_rev": "19f804ba738ca0e6133bc19d7ade010208c9a8db", "commit_rev":
"e9ee3d63b09c9ef3f9dafc84bf7cfa83c89203fc"}

[commit-bot: I haz the power, 2017-03-17 22:22:37]

Description was changed from

==========
Client config and QUIC transaction integration tests

BUG=680570, 680564
==========

to

==========
Client config and QUIC transaction integration tests

BUG=680570, 680564

Review-Url: https://codereview.chromium.org/2754803004
Cr-Commit-Position: refs/heads/master@{#457892}
Committed:
https://chromium.googlesource.com/chromium/src/+/e9ee3d63b09c9ef3f9dafc84bf7c...
==========

[commit-bot: I haz the power, 2017-03-17 22:22:38]

Committed patchset #1 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/e9ee3d63b09c9ef3f9dafc84bf7c...