plumb network upload into the task manager

chrome/browser/task_manager/providers/browser_process_task_unittest.cc

Index: chrome/browser/task_manager/providers/browser_process_task_unittest.cc
diff --git a/chrome/browser/task_manager/providers/browser_process_task_unittest.cc b/chrome/browser/task_manager/providers/browser_process_task_unittest.cc
index 15fbc672b1a190b0e708b64b4869eb804a9380f2..8cfbbb23c2babb96d5a8c6e1f4c45d3a460fb129 100644
--- a/chrome/browser/task_manager/providers/browser_process_task_unittest.cc
+++ b/chrome/browser/task_manager/providers/browser_process_task_unittest.cc
@@ -75,14 +75,11 @@ TEST_F(BrowserProcessTaskProviderTest, TestProvidedTask) {
   EXPECT_EQ(Task::BROWSER, provided_task_->GetType());
   EXPECT_EQ(0, provided_task_->GetChildProcessUniqueID());
   const int received_bytes = 1024;
-  EXPECT_FALSE(provided_task_->ReportsNetworkUsage());
-  EXPECT_EQ(-1, provided_task_->network_usage());
+  EXPECT_EQ(0, provided_task_->network_usage_rate());
   provided_task_->OnNetworkBytesRead(received_bytes);
   // Do a refresh with a 1-second update time.
   provided_task_->Refresh(base::TimeDelta::FromSeconds(1),
                           REFRESH_TYPE_NETWORK_USAGE);

[ncarter (slow), 2017/06/16 22:28:30]

Now that the main tests of network-usage functionality have moved out of this
class, it makes sense to restore the original assertion here:

  EXPECT_EQ(received_bytes, provided_task_->network_usage_rate());

It works as a "sanity check that the browser process can show network usage".

[cburn, 2017/06/19 22:07:08]

Done.

-  EXPECT_TRUE(provided_task_->ReportsNetworkUsage());
-  EXPECT_EQ(received_bytes, provided_task_->network_usage());
 }
 
 }  // namespace task_manager