Some required fixes for HTMLViewer rotate

android_webview/native/state_serializer.cc

 // Copyright (c) 2012 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 "android_webview/native/state_serializer.h"
 
 #include <string>
 
 #include "base/memory/scoped_vector.h"
 #include "base/pickle.h"
@@ skipping 80 matching lines @@
     return false;
   if (selected_entry >= entry_count)
     return false;
 
   ScopedVector<content::NavigationEntry> restored_entries;
   for (int i = 0; i < entry_count; ++i) {
     restored_entries.push_back(content::NavigationEntry::Create());
     if (!internal::RestoreNavigationEntryFromPickle(iterator,
                                                     restored_entries[i]))
       return false;
+
+    restored_entries[i]->SetPageID(i);

[joth, 2012/11/29 05:56:14]

should we be storing  GetPageID in saveState and restore it here? (if chrome 
already does it this way, then fair enough)

[boliu, 2012/11/29 06:14:03]

PageID is incrementing counter in NavigationController for each new page
navigated in the controller. No need to save old values since it might have
gaps. This is what chrome does as well.

   }
 
   // |web_contents| takes ownership of these entries after this call.
   web_contents->GetController().Restore(
       selected_entry,
       content::NavigationController::RESTORE_LAST_SESSION_EXITED_CLEANLY,
       &restored_entries.get());
   DCHECK_EQ(0u, restored_entries.size());
 
   return true;
@@ skipping 135 matching lines @@
       return false;
     entry->SetTimestamp(base::Time::FromInternalValue(timestamp));
   }
 
   return true;
 }
 
 }  // namespace internal
 
 }  // namespace android_webview