plumb network upload into the task manager

chrome/browser/task_manager/providers/browser_process_task_unittest.cc

 // Copyright 2015 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 "base/macros.h"
 #include "chrome/browser/task_manager/providers/browser_process_task_provider.h"
 #include "chrome/browser/task_manager/task_manager_observer.h"
 #include "chrome/grit/generated_resources.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/base/l10n/l10n_util.h"
@@ skipping 57 matching lines @@
   ASSERT_NE(nullptr, provided_task_);
 
   EXPECT_EQ(base::GetCurrentProcessHandle(), provided_task_->process_handle());
   EXPECT_EQ(base::GetCurrentProcId(), provided_task_->process_id());
   EXPECT_FALSE(provided_task_->ReportsWebCacheStats());
   EXPECT_EQ(l10n_util::GetStringUTF16(IDS_TASK_MANAGER_WEB_BROWSER_CELL_TEXT),
             provided_task_->title());
   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