This is to follow up on a review comment: Change this switch to include all values in the enum and not have a default label, so that compilers errors occur here when the enum is changed without the switch being changed?
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/666c068451e48641d613f4470bb7a151b3ca21d8 commit 666c068451e48641d613f4470bb7a151b3ca21d8 Author: Helen Li <xunjieli@chromium.org> Date: Fri Sep 15 19:57:05 2017 Deprecate SDCH-related net::SourceStream types - Deprecated three SDCH-related net::SourceStream types - Modify URLRequestHttpJob::SetUpSourceStream() to not include default case Bug: 764434 , 764433 , 762686 Change-Id: I93320bfee930484da76a0a29fab2224f2b886523 Reviewed-on: https://chromium-review.googlesource.com/667731 Commit-Queue: Helen Li <xunjieli@chromium.org> Reviewed-by: Randy Smith <rdsmith@chromium.org> Cr-Commit-Position: refs/heads/master@{#502337} [modify] https://crrev.com/666c068451e48641d613f4470bb7a151b3ca21d8/net/filter/filter_source_stream.cc [modify] https://crrev.com/666c068451e48641d613f4470bb7a151b3ca21d8/net/filter/gzip_source_stream.cc [modify] https://crrev.com/666c068451e48641d613f4470bb7a151b3ca21d8/net/filter/gzip_source_stream.h [modify] https://crrev.com/666c068451e48641d613f4470bb7a151b3ca21d8/net/filter/gzip_source_stream_fuzzer.cc [modify] https://crrev.com/666c068451e48641d613f4470bb7a151b3ca21d8/net/filter/gzip_source_stream_unittest.cc [modify] https://crrev.com/666c068451e48641d613f4470bb7a151b3ca21d8/net/filter/source_stream_type_list.h [modify] https://crrev.com/666c068451e48641d613f4470bb7a151b3ca21d8/net/url_request/url_request_http_job.cc [modify] https://crrev.com/666c068451e48641d613f4470bb7a151b3ca21d8/net/url_request/url_request_http_job_unittest.cc
Comment 1 by xunji...@chromium.org
, Sep 12 2017