Change Lo-Fi bool to bitmask to support multiple Previews types

chrome/browser/data_use_measurement/chrome_data_use_ascriber_unittest.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/data_use_measurement/chrome_data_use_ascriber.h"
 
 #include <list>
 #include <memory>
 
 #include "base/message_loop/message_loop.h"
 #include "base/run_loop.h"
 #include "components/data_use_measurement/core/data_use_recorder.h"
 #include "content/public/browser/resource_request_info.h"
 #include "content/public/common/browser_side_navigation_policy.h"
+#include "content/public/common/previews_state.h"
 #include "content/public/common/process_type.h"
 #include "content/public/test/mock_resource_context.h"
 #include "content/public/test/test_browser_thread_bundle.h"
 #include "net/url_request/url_request_test_util.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace {
 int kRenderProcessId = 1;
 int kRenderFrameId = 2;
 int kRequestId = 3;
@@ skipping 25 matching lines @@
 
   std::unique_ptr<net::URLRequest> CreateNewRequest(std::string url,
                                                     bool is_main_frame,
                                                     int request_id,
                                                     int render_process_id,
                                                     int render_frame_id) {
     std::unique_ptr<net::URLRequest> request =
         context()->CreateRequest(GURL(url), net::IDLE, nullptr);
     // TODO(kundaji): Allow request_id to be specified in AllocateForTesting.
     content::ResourceRequestInfo::AllocateForTesting(
-        request.get(),
-        content::RESOURCE_TYPE_MAIN_FRAME,
-        resource_context(),
+        request.get(), content::RESOURCE_TYPE_MAIN_FRAME, resource_context(),
         render_process_id,
         -1,  // render_view_id
-        render_frame_id,
-        is_main_frame,
-        false,   // parent_is_main_frame
-        false,   // allow_download
-        true,    // is_async
-        false);  // is_using_lofi
+        render_frame_id, is_main_frame,
+        false,                   // parent_is_main_frame
+        false,                   // allow_download
+        true,                    // is_async
+        content::PREVIEWS_OFF);  // previews_state

[Nico, 2017/01/09 22:35:03]

nit: remove now-useless comment for PREVIEWS_OFF

[megjablon, 2017/01/10 22:30:01]

Done.

     return request;
   }
 
  private:
   content::TestBrowserThreadBundle thread_bundle_;
   ChromeDataUseAscriber ascriber_;
   net::TestURLRequestContext context_;
   std::unique_ptr<content::MockResourceContext> resource_context_;
 };
 
@@ skipping 14 matching lines @@
   EXPECT_EQ(1u, recorders().size());
 
   // Request should cause a recorder to be created.
   ascriber()->OnBeforeUrlRequest(request.get());
   EXPECT_EQ(2u, recorders().size());
 
   ascriber()->RenderFrameDeleted(kRenderProcessId, kRenderFrameId, -1, -1);
 }
 
 }  // namespace data_use_measurement