GitLab

The QuicStreamFactory methods definition order was completely different
from the declaration order in the header. This CL fixes that. It also
renames the test-only method CryptoConfigCacheIsEmpty() to
CryptoConfigCacheIsEmptyForTesting().

This CL doesn't actually change any code, other than the rename and
moving stuff around.

Bug: 1003423
Change-Id: I751d656cf24b41e206414be04958ac52cf4a9faf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1819352Reviewed-by: Zhongyi Shi <zhongyi@chromium.org>
Commit-Queue: Matt Menke <mmenke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#698980}