Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(228)

Side by Side Diff: content/browser/loader/mojo_async_resource_handler_unittest.cc

Issue 2484633004: Change Lo-Fi bool to bitmask to support multiple Previews types (Closed)
Patch Set: add back previews_unspecified Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/loader/mojo_async_resource_handler.h" 5 #include "content/browser/loader/mojo_async_resource_handler.h"
6 6
7 #include <string.h> 7 #include <string.h>
8 8
9 #include <utility> 9 #include <utility>
10 10
(...skipping 123 matching lines...) Expand 10 before | Expand all | Expand 10 after
134 net::URLRequest* request, 134 net::URLRequest* request,
135 ResourceContext* resource_context, 135 ResourceContext* resource_context,
136 ResourceResponse* response) override { 136 ResourceResponse* response) override {
137 ADD_FAILURE() << "OnRequestRedirected should not be called."; 137 ADD_FAILURE() << "OnRequestRedirected should not be called.";
138 } 138 }
139 139
140 void RequestComplete(net::URLRequest* url_request) override { 140 void RequestComplete(net::URLRequest* url_request) override {
141 ADD_FAILURE() << "RequestComplete should not be called."; 141 ADD_FAILURE() << "RequestComplete should not be called.";
142 } 142 }
143 143
144 bool ShouldEnableLoFiMode( 144 int GetPreviewsState(
nasko 2016/12/08 22:19:32 PreviewsState
megjablon 2016/12/09 20:35:54 Done.
145 const net::URLRequest& url_request, 145 const net::URLRequest& url_request,
146 content::ResourceContext* resource_context) override { 146 content::ResourceContext* resource_context) override {
147 ADD_FAILURE() << "ShouldEnableLoFiMode should not be called."; 147 ADD_FAILURE() << "GetPreviewsState should not be called.";
148 return false; 148 return 0;
nasko 2016/12/08 22:19:32 Use the symbolic name - PREVIEWS_UNSPECIFIED or PR
megjablon 2016/12/09 20:35:54 Done.
149 } 149 }
150 150
151 NavigationData* GetNavigationData(net::URLRequest* request) const override { 151 NavigationData* GetNavigationData(net::URLRequest* request) const override {
152 ADD_FAILURE() << "GetNavigationData should not be called."; 152 ADD_FAILURE() << "GetNavigationData should not be called.";
153 return nullptr; 153 return nullptr;
154 } 154 }
155 155
156 std::unique_ptr<net::ClientCertStore> CreateClientCertStore( 156 std::unique_ptr<net::ClientCertStore> CreateClientCertStore(
157 ResourceContext* resource_context) override { 157 ResourceContext* resource_context) override {
158 ADD_FAILURE() << "CreateClientCertStore should not be called."; 158 ADD_FAILURE() << "CreateClientCertStore should not be called.";
(...skipping 1111 matching lines...) Expand 10 before | Expand all | Expand 10 after
1270 } 1270 }
1271 } 1271 }
1272 EXPECT_EQ("B", body); 1272 EXPECT_EQ("B", body);
1273 } 1273 }
1274 1274
1275 INSTANTIATE_TEST_CASE_P(MojoAsyncResourceHandlerWithAllocationSizeTest, 1275 INSTANTIATE_TEST_CASE_P(MojoAsyncResourceHandlerWithAllocationSizeTest,
1276 MojoAsyncResourceHandlerWithAllocationSizeTest, 1276 MojoAsyncResourceHandlerWithAllocationSizeTest,
1277 ::testing::Values(8, 32 * 2014)); 1277 ::testing::Values(8, 32 * 2014));
1278 } // namespace 1278 } // namespace
1279 } // namespace content 1279 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698