Simplify EncodePageState to return no value.

content/common/page_state_serialization_unittest.cc

 // Copyright 2013 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 <stddef.h>
 
 #include <cmath>
 
 #include "base/base64.h"
 #include "base/files/file_util.h"
@@ skipping 223 matching lines @@
     PopulatePageStateForBackwardsCompatTest(&expected);
 
     ExpectEquality(expected, output);
   }
 };
 
 TEST_F(PageStateSerializationTest, BasicEmpty) {
   ExplodedPageState input;
 
   std::string encoded;
-  EXPECT_TRUE(EncodePageState(input, &encoded));
+  EncodePageState(input, &encoded);
 
   ExplodedPageState output;
   EXPECT_TRUE(DecodePageState(encoded, &output));
 
   ExpectEquality(input, output);
 }
 
 TEST_F(PageStateSerializationTest, BasicFrame) {
   ExplodedPageState input;
   PopulateFrameState(&input.top);
 
   std::string encoded;
-  EXPECT_TRUE(EncodePageState(input, &encoded));
+  EncodePageState(input, &encoded);
 
   ExplodedPageState output;
   EXPECT_TRUE(DecodePageState(encoded, &output));
 
   ExpectEquality(input, output);
 }
 
 TEST_F(PageStateSerializationTest, BasicFramePOST) {
   ExplodedPageState input;
   PopulateFrameState(&input.top);
   PopulateHttpBody(&input.top.http_body, &input.referenced_files);
 
   std::string encoded;
-  EXPECT_TRUE(EncodePageState(input, &encoded));
+  EncodePageState(input, &encoded);
 
   ExplodedPageState output;
   EXPECT_TRUE(DecodePageState(encoded, &output));
 
   ExpectEquality(input, output);
 }
 
 TEST_F(PageStateSerializationTest, BasicFrameSet) {
   ExplodedPageState input;
   PopulateFrameState(&input.top);
 
   // Add some child frames.
   for (int i = 0; i < 4; ++i) {
     ExplodedFrameState child_state;
     PopulateFrameState(&child_state);
     input.top.children.push_back(child_state);
   }
 
   std::string encoded;
-  EXPECT_TRUE(EncodePageState(input, &encoded));
+  EncodePageState(input, &encoded);
 
   ExplodedPageState output;
   EXPECT_TRUE(DecodePageState(encoded, &output));
 
   ExpectEquality(input, output);
 }
 
 TEST_F(PageStateSerializationTest, BasicFrameSetPOST) {
   ExplodedPageState input;
   PopulateFrameState(&input.top);
@@ skipping 62 matching lines @@
 }
 
 TEST_F(PageStateSerializationTest, DumpExpectedPageStateForBackwardsCompat) {
   // Change to #if 1 to enable this code.  Use this code to generate data, based
   // on the current serialization format, for the BackwardsCompat_vXX tests.
 #if 0
   ExplodedPageState state;
   PopulatePageStateForBackwardsCompatTest(&state);
 
   std::string encoded;
-  EXPECT_TRUE(EncodePageState(state, &encoded));
+  EncodePageState(state, &encoded);
 
   std::string base64;
   base::Base64Encode(encoded, &base64);
 
   base::FilePath path;
   PathService::Get(base::DIR_TEMP, &path);
   path = path.AppendASCII("expected.dat");
 
   FILE* fp = base::OpenFile(path, "wb");
   ASSERT_TRUE(fp);
@@ skipping 49 matching lines @@
 TEST_F(PageStateSerializationTest, BackwardsCompat_v21) {
   TestBackwardsCompat(21);
 }
 
 TEST_F(PageStateSerializationTest, BackwardsCompat_v22) {
   TestBackwardsCompat(22);
 }
 
 }  // namespace
 }  // namespace content