Clean up and modernize the feedback code

components/feedback/feedback_common_unittest.cc

 // Copyright 2014 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/feedback/feedback_common.h"
 
 #include "base/bind.h"
 #include "components/feedback/proto/common.pb.h"
 #include "components/feedback/proto/dom.pb.h"
 #include "components/feedback/proto/extension.pb.h"
@@ skipping 19 matching lines @@
 }  // namespace
 
 class FeedbackCommonTest : public testing::Test {
  protected:
   FeedbackCommonTest()
       : feedback_(new FeedbackCommon()) {
   }
 
   ~FeedbackCommonTest() override {}
 
+  void CompressLogs() { feedback_->CompressLogs(); }
+
   bool FeedbackHasProductId() const { return feedback_->HasProductId(); }
 
   scoped_refptr<FeedbackCommon> feedback_;
   userfeedback::ExtensionSubmit report_;
 };
 
 TEST_F(FeedbackCommonTest, TestBasicData) {
   // Test that basic data can be set and propagates to the request.
   feedback_->set_category_tag(kOne);
   feedback_->set_description(kTwo);
@@ skipping 35 matching lines @@
 
   EXPECT_EQ(1, report_.web_data().product_specific_data_size());
   EXPECT_EQ(kOne, report_.web_data().product_specific_data(0).key());
 }
 
 TEST_F(FeedbackCommonTest, TestCompression) {
   // Add a large and small log, verify that an attachment has been
   // added with the right name.
   feedback_->AddLog(kOne, kTwo);
   feedback_->AddLog(kThree, kLongLog);
-  feedback_->CompressLogs();
+  CompressLogs();
   feedback_->PrepareReport(&report_);
 
   EXPECT_EQ(1, report_.product_specific_binary_data_size());
   EXPECT_EQ(kLogsAttachmentName,
             report_.product_specific_binary_data(0).name());
 }