Return operation name in prefetch request callback and add internal page hookup

components/offline_pages/core/prefetch/generate_page_bundle_request_unittest.cc

 // Copyright 2017 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 "components/offline_pages/core/prefetch/generate_page_bundle_request.h"
 
 #include "base/test/mock_callback.h"
 #include "components/offline_pages/core/prefetch/prefetch_request_test_base.h"
 #include "components/offline_pages/core/prefetch/prefetch_types.h"
 #include "components/offline_pages/core/prefetch/proto/offline_pages.pb.h"
@@ skipping 59 matching lines @@
   EXPECT_EQ(kTestURL2, bundle_data.pages(1).url());
   EXPECT_EQ(proto::NO_TRANSFORMATION, bundle_data.pages(1).transformation());
 }
 
 TEST_F(GeneratePageBundleRequestTest, EmptyResponse) {
   base::MockCallback<PrefetchRequestFinishedCallback> callback;
   std::unique_ptr<GeneratePageBundleRequest> request(
       CreateRequest(callback.Get()));
 
   PrefetchRequestStatus status;
+  std::string operation_name;
   std::vector<RenderPageInfo> pages;
-  EXPECT_CALL(callback, Run(_, _))
-      .WillOnce(DoAll(SaveArg<0>(&status), SaveArg<1>(&pages)));
+  EXPECT_CALL(callback, Run(_, _, _))
+      .WillOnce(DoAll(SaveArg<0>(&status), SaveArg<1>(&operation_name),
+                      SaveArg<2>(&pages)));
   RespondWithData("");
 
   EXPECT_EQ(PrefetchRequestStatus::SHOULD_RETRY_WITH_BACKOFF, status);
+  EXPECT_TRUE(operation_name.empty());
   EXPECT_TRUE(pages.empty());
 }
 
 TEST_F(GeneratePageBundleRequestTest, InvalidResponse) {
   base::MockCallback<PrefetchRequestFinishedCallback> callback;
   std::unique_ptr<GeneratePageBundleRequest> request(
       CreateRequest(callback.Get()));
 
   PrefetchRequestStatus status;
+  std::string operation_name;
   std::vector<RenderPageInfo> pages;
-  EXPECT_CALL(callback, Run(_, _))
-      .WillOnce(DoAll(SaveArg<0>(&status), SaveArg<1>(&pages)));
+  EXPECT_CALL(callback, Run(_, _, _))
+      .WillOnce(DoAll(SaveArg<0>(&status), SaveArg<1>(&operation_name),
+                      SaveArg<2>(&pages)));
   RespondWithData("Some invalid data");
 
   EXPECT_EQ(PrefetchRequestStatus::SHOULD_RETRY_WITH_BACKOFF, status);
+  EXPECT_TRUE(operation_name.empty());
   EXPECT_TRUE(pages.empty());
 }
 
 }  // namespace offline_pages