Merge TestResourceHandlers used by MimeSniffingRH and InterceptingRH tests |
|||
Issue descriptionThey do basically the same thing. Merging them would make the tests easier to read, and ensure they support the same feature sets, which makes updating tests to do new things easier.
,
Oct 25 2016
,
Oct 27 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/2b6880fff5eb65c0d7f12fa3b755aaac22bf13a9 commit 2b6880fff5eb65c0d7f12fa3b755aaac22bf13a9 Author: mmenke <mmenke@chromium.org> Date: Thu Oct 27 09:42:06 2016 Merge TestResourceHandlers used by two different test fixtures. The MimeSniffingResourceHandler and InterceptingResourceHandler unit tests use two different TestResourceHandlers with very similar feature sets. This CL merges them into single class that supports all the features both text fixtures need. BUG= 658760 Review-Url: https://codereview.chromium.org/2441293003 Cr-Commit-Position: refs/heads/master@{#427981} [modify] https://crrev.com/2b6880fff5eb65c0d7f12fa3b755aaac22bf13a9/content/browser/loader/intercepting_resource_handler_unittest.cc [modify] https://crrev.com/2b6880fff5eb65c0d7f12fa3b755aaac22bf13a9/content/browser/loader/mime_sniffing_resource_handler.cc [modify] https://crrev.com/2b6880fff5eb65c0d7f12fa3b755aaac22bf13a9/content/browser/loader/mime_sniffing_resource_handler_unittest.cc [add] https://crrev.com/2b6880fff5eb65c0d7f12fa3b755aaac22bf13a9/content/browser/loader/test_resource_handler.cc [add] https://crrev.com/2b6880fff5eb65c0d7f12fa3b755aaac22bf13a9/content/browser/loader/test_resource_handler.h [modify] https://crrev.com/2b6880fff5eb65c0d7f12fa3b755aaac22bf13a9/content/test/BUILD.gn
,
Oct 27 2016
|
|||
►
Sign in to add a comment |
|||
Comment 1 by mmenke@chromium.org
, Oct 24 2016