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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: content/browser/loader/async_resource_handler_unittest.cc

Issue 2484633004: Change Lo-Fi bool to bitmask to support multiple Previews types (Closed)
Patch Set: use PreviewsState everywhere 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 side-by-side diff with in-line comments
Download patch
« content/browser/frame_host/navigator_impl.cc ('K') | « content/browser/loader/DEPS ('k') | content/browser/loader/intercepting_resource_handler_unittest.cc » ('j') | content/browser/loader/resource_dispatcher_host_browsertest.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/loader/async_resource_handler_unittest.cc
diff --git a/content/browser/loader/async_resource_handler_unittest.cc b/content/browser/loader/async_resource_handler_unittest.cc
index 002e38b6f33e803c6df63f97114cd1f5f6e8fd5e..80ed47814b722324e534c323e70d680609b835d7 100644
--- a/content/browser/loader/async_resource_handler_unittest.cc
+++ b/content/browser/loader/async_resource_handler_unittest.cc
@@ -33,6 +33,7 @@
#include "content/public/browser/resource_context.h"
#include "content/public/browser/resource_request_info.h"
#include "content/public/common/content_features.h"
+#include "content/public/common/previews_state.h"
#include "content/public/common/resource_type.h"
#include "content/public/test/mock_resource_context.h"
#include "content/public/test/test_browser_thread_bundle.h"
@@ -187,7 +188,7 @@ class AsyncResourceHandlerTest : public ::testing::Test,
resource_context_.get(), // context
false, // report_raw_headers
true, // is_async
- false, // is_using_lofi
+ PREVIEWS_OFF, // previews_state
nasko 2016/12/12 23:05:41 nit: Needs to be aligned with the rest.
megjablon 2016/12/13 03:03:14 Done.
std::string(), // original_headers
nullptr, // body
false); // initiated_in_secure_context
« content/browser/frame_host/navigator_impl.cc ('K') | « content/browser/loader/DEPS ('k') | content/browser/loader/intercepting_resource_handler_unittest.cc » ('j') | content/browser/loader/resource_dispatcher_host_browsertest.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698