Simplify WebContentsState::GetContentsStateAsByteBuffer

chrome/browser/android/tab_state.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 "chrome/browser/android/tab_state.h"
 
 #include <jni.h>
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 268 matching lines @@
   if (*current_entry_index < 0 ||
       *current_entry_index >= static_cast<int>(navigations->size()))
     return false;
 
   return true;
 }
 
 };  // anonymous namespace
 
 ScopedJavaLocalRef<jobject> WebContentsState::GetContentsStateAsByteBuffer(
-    JNIEnv* env, TabAndroid* tab) {
+    JNIEnv* env,
+    TabAndroid* tab) {
   Profile* profile = tab->GetProfile();
   if (!profile)
     return ScopedJavaLocalRef<jobject>();
 
   content::NavigationController& controller =
       tab->web_contents()->GetController();
-  const int pending_index = controller.GetPendingEntryIndex();
-  int entry_count = controller.GetEntryCount();
-  if (entry_count == 0 && pending_index == 0)

[Charlie Reis, 2017/04/11 23:31:08]

Right.  pending_index should be -1 if entry_count is 0.

-    entry_count++;
-
+  const int entry_count = controller.GetEntryCount();
   if (entry_count == 0)
     return ScopedJavaLocalRef<jobject>();
 
-  int current_entry = controller.GetLastCommittedEntryIndex();
-  if (current_entry == -1 && entry_count > 0)
-    current_entry = 0;

[Charlie Reis, 2017/04/11 23:31:08]

Yeah, I think this was in response to the entry_count++ line, which shouldn't
have been reachable.

-
   std::vector<content::NavigationEntry*> navigations(entry_count);
   for (int i = 0; i < entry_count; ++i) {
-    content::NavigationEntry* entry = (i == pending_index) ?
-        controller.GetPendingEntry() : controller.GetEntryAtIndex(i);
-    navigations[i] = entry;
+    navigations[i] = controller.GetEntryAtIndex(i);
   }
 
   return WebContentsState::WriteNavigationsAsByteBuffer(
-      env,
-      profile->IsOffTheRecord(),
-      navigations,
-      current_entry);
+      env, profile->IsOffTheRecord(), navigations,
+      controller.GetLastCommittedEntryIndex());
 }
 
 // Common implementation for GetContentsStateAsByteBuffer() and
 // CreateContentsStateAsByteBuffer(). Does not assume ownership of the
 // navigations.
 ScopedJavaLocalRef<jobject> WebContentsState::WriteNavigationsAsByteBuffer(
     JNIEnv* env,
     bool is_off_the_record,
     const std::vector<content::NavigationEntry*>& navigations,
     int current_entry) {
@@ skipping 242 matching lines @@
   std::unique_ptr<WebContents> web_contents(WebContents::FromJavaWebContents(
       WebContentsState::RestoreContentsFromByteBuffer(
           env, clazz, state, saved_state_version, true)));
   if (web_contents.get())
     TabAndroid::CreateHistoricalTabFromContents(web_contents.get());
 }
 
 bool RegisterTabState(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }