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

Unified Diff: chrome/browser/safe_browsing/incident_reporting/resource_request_detector_unittest.cc

Issue 2484633004: Change Lo-Fi bool to bitmask to support multiple Previews types (Closed)
Patch Set: fix ContentResourceProviderTest Created 3 years, 11 months 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
Index: chrome/browser/safe_browsing/incident_reporting/resource_request_detector_unittest.cc
diff --git a/chrome/browser/safe_browsing/incident_reporting/resource_request_detector_unittest.cc b/chrome/browser/safe_browsing/incident_reporting/resource_request_detector_unittest.cc
index 838720be1db3e85a8c0d85a8237d943d2541c627..cbaf7127617e4495054210e7df6a5da376b3f765 100644
--- a/chrome/browser/safe_browsing/incident_reporting/resource_request_detector_unittest.cc
+++ b/chrome/browser/safe_browsing/incident_reporting/resource_request_detector_unittest.cc
@@ -15,6 +15,7 @@
#include "chrome/common/safe_browsing/csd.pb.h"
#include "components/safe_browsing_db/test_database_manager.h"
#include "content/public/browser/resource_request_info.h"
+#include "content/public/common/previews_state.h"
#include "content/public/common/resource_type.h"
#include "content/public/test/test_browser_thread_bundle.h"
#include "crypto/sha2.h"
@@ -90,15 +91,14 @@ class ResourceRequestDetectorTest : public testing::Test {
content::ResourceRequestInfo::AllocateForTesting(
url_request.get(), resource_type,
- NULL, // resource_context
- 0, // render_process_id
- 0, // render_view_id
- MSG_ROUTING_NONE, // render_frame_id
- true, // is_main_frame
- false, // parent_is_main_frame
- true, // allow_download
- false, // is_async
- false); // is_using_lofi
+ /*resource_context=*/NULL,
+ /*render_process_id=*/0,
+ /*render_view_id=*/0,
+ /*render_frame_id=*/MSG_ROUTING_NONE,
+ /*is_main_frame=*/true,
+ /*parent_is_main_frame=*/false,
+ /*allow_download=*/true,
+ /*is_async=*/false, content::PREVIEWS_OFF);
return url_request;
}
« no previous file with comments | « chrome/browser/prerender/prerender_resource_throttle_unittest.cc ('k') | chrome/browser/search/iframe_source_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698