Simplify EncodePageState to return no value.

content/public/common/page_state.cc

 // Copyright (c) 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 "content/public/common/page_state.h"
 
 #include <stddef.h>
 
 #include "base/files/file_path.h"
 #include "base/strings/nullable_string16.h"
@@ skipping 14 matching lines @@
 void ToFilePathVector(const std::vector<base::NullableString16>& input,
                       std::vector<base::FilePath>* output) {
   output->clear();
   output->reserve(input.size());
   for (size_t i = 0; i < input.size(); ++i)
     output->push_back(ToFilePath(input[i]));
 }
 
 PageState ToPageState(const ExplodedPageState& state) {
   std::string encoded_data;
-  if (!EncodePageState(state, &encoded_data))
-    return PageState();
-
+  EncodePageState(state, &encoded_data);
   return PageState::CreateFromEncodedData(encoded_data);
 }
 
 void RecursivelyRemovePasswordData(ExplodedFrameState* state) {
   if (state->http_body.contains_passwords)
     state->http_body = ExplodedHttpBody();
 }
 
 void RecursivelyRemoveScrollOffset(ExplodedFrameState* state) {
   state->scroll_offset = gfx::Point();
@@ skipping 118 matching lines @@
 }
 
 PageState::PageState(const std::string& data)
     : data_(data) {
   // TODO(darin): Enable this DCHECK once tests have been fixed up to not pass
   // bogus encoded data to CreateFromEncodedData.
   //DCHECK(IsValid());
 }
 
 }  // namespace content