Adding DRP specfic UMA for FirstContentfulPaint

chrome/browser/renderer_host/chrome_navigation_data_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/renderer_host/chrome_navigation_data.h"
+
+#include <memory>
+
+#include "base/memory/ptr_util.h"
+#include "components/data_reduction_proxy/core/browser/data_reduction_proxy_data.h"
+#include "content/public/browser/navigation_data.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+class ChromeNavigationDataTest : public testing::Test {
+ public:
+  ChromeNavigationDataTest() {}
+  ~ChromeNavigationDataTest() override {}
+};
+
+TEST_F(ChromeNavigationDataTest, AddingDataReductionProxyData) {
+  std::unique_ptr<ChromeNavigationData> data(new ChromeNavigationData());
+  data_reduction_proxy::DataReductionProxyData* data_reduction_proxy_data =
+      new data_reduction_proxy::DataReductionProxyData();
+  data->SetDataReductionProxyData(base::WrapUnique(data_reduction_proxy_data));
+  EXPECT_EQ(data_reduction_proxy_data, data->GetDataReductionProxyData());
+}
+
+TEST_F(ChromeNavigationDataTest, Clone) {
+  std::unique_ptr<ChromeNavigationData> data(new ChromeNavigationData());
+  data_reduction_proxy::DataReductionProxyData* data_reduction_proxy_data =
+      new data_reduction_proxy::DataReductionProxyData();
+  data->SetDataReductionProxyData(base::WrapUnique(data_reduction_proxy_data));
+  std::unique_ptr<content::NavigationData> clone_data = data->Clone();
+  ChromeNavigationData* clone_chrome_data =
+      static_cast<ChromeNavigationData*>(clone_data.get());
+  EXPECT_NE(data.get(), clone_data.get());
+  EXPECT_NE(data.get(), clone_chrome_data);
+  EXPECT_NE(data->GetDataReductionProxyData(),
+            clone_chrome_data->GetDataReductionProxyData());
+  EXPECT_EQ(data_reduction_proxy_data, data->GetDataReductionProxyData());
+  EXPECT_FALSE(!clone_chrome_data->GetDataReductionProxyData());

[nasko, 2016/05/09 17:35:43]

Why not EXPECT_TRUE?

[RyanSturm, 2016/05/09 18:27:31]

Done.

+}