Adding DRP specfic UMA for FirstContentfulPaint

chrome/browser/renderer_host/chrome_navigation_data.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 "base/memory/ptr_util.h"
+#include "components/data_reduction_proxy/core/browser/data_reduction_proxy_data.h"
+
+ChromeNavigationData::ChromeNavigationData() {}
+
+ChromeNavigationData::~ChromeNavigationData() {}
+
+std::unique_ptr<content::NavigationData> ChromeNavigationData::Clone() const {
+  ChromeNavigationData* copy = new ChromeNavigationData();

[nasko, 2016/04/26 20:18:39]

Why keep a raw pointer? Just use the unique_ptr.

[RyanSturm, 2016/04/27 23:27:46]

Done.

+  if (data_reduction_proxy_data_)
+    copy->SetDataReductionProxyData(data_reduction_proxy_data_->DeepCopy());
+  return base::WrapUnique(copy);

[nasko, 2016/04/26 20:18:39]

You should WrapUnique the return value of the allocation.

[RyanSturm, 2016/04/27 23:27:46]

Done.

+}