Store AppItems as pages in AppListModel.

chrome/browser/ui/app_list/fast_show_pickler.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 "chrome/browser/ui/app_list/fast_show_pickler.h"
 
 #include "third_party/skia/include/core/SkBitmap.h"
 #include "ui/app_list/app_list_item_model.h"
 #include "ui/gfx/image/image_skia_rep.h"
 
@@ skipping 171 matching lines @@
     return false;
   if (!pickle->WriteString(item->full_name()))
     return false;
   if (!pickle->WriteBool(item->has_shadow()))
     return false;
   if (!PickleImage(pickle, item->icon()))
     return false;
   return true;
 }
 
-void CopyOverItem(AppListItemModel* src_item, AppListItemModel* dest_item) {
-  dest_item->SetTitleAndFullName(src_item->title(), src_item->full_name());
-  dest_item->SetIcon(src_item->icon(), src_item->has_shadow());
-}
-
 }  // namespace
 
 // The version of the pickle format defined here. This needs to be incremented
 // whenever this format is changed so new clients can invalidate old versions.
-const int FastShowPickler::kVersion = 1;
+const int FastShowPickler::kVersion = 2;
 
 scoped_ptr<Pickle> FastShowPickler::PickleAppListModelForFastShow(
     AppListModel* model) {
   scoped_ptr<Pickle> result(new Pickle);
   if (!result->WriteInt(kVersion))
     return scoped_ptr<Pickle>();
   if (!result->WriteBool(model->signed_in()))
     return scoped_ptr<Pickle>();
-  if (!result->WriteInt((int) model->apps()->item_count()))
+  // Write the number of pages.
+  size_t page_count = model->GetNumAppPages();
+  if (!result->WriteInt(static_cast<int>(page_count)))
     return scoped_ptr<Pickle>();
-  for (size_t i = 0; i < model->apps()->item_count(); ++i) {
-    if (!PickleAppListItemModel(result.get(), model->apps()->GetItemAt(i)))
+  for (size_t p = 0; p < page_count; ++p) {
+    // Write the number of items in the page.
+    size_t item_count = model->GetAppItems(p).item_count();
+    if (!result->WriteInt(static_cast<int>(item_count)))
       return scoped_ptr<Pickle>();
+    // Write the items in the page.
+    for (size_t i = 0; i < item_count; ++i) {
+      if (!PickleAppListItemModel(result.get(), model->GetItemAt(p, i)))
+        return scoped_ptr<Pickle>();
+    }
   }
   return result.Pass();
 }
 
-void FastShowPickler::CopyOver(AppListModel* src, AppListModel* dest) {
-  dest->apps()->DeleteAll();
-  dest->SetSignedIn(src->signed_in());
-  for (size_t i = 0; i < src->apps()->item_count(); i++) {
-    AppListItemModel* src_item = src->apps()->GetItemAt(i);
-    AppListItemModel* dest_item = new AppListItemModel(src_item->id());
-    CopyOverItem(src_item, dest_item);
-    dest->apps()->Add(dest_item);
-  }
-}
-
 scoped_ptr<AppListModel>
 FastShowPickler::UnpickleAppListModelForFastShow(Pickle* pickle) {
   PickleIterator it(*pickle);
   int read_version = 0;
   if (!it.ReadInt(&read_version))
     return scoped_ptr<AppListModel>();
   if (read_version != kVersion)
     return scoped_ptr<AppListModel>();
-  int app_count = 0;
   bool signed_in = false;
   if (!it.ReadBool(&signed_in))
     return scoped_ptr<AppListModel>();
-  if (!it.ReadInt(&app_count))
+  // Read the number of pages.
+  int page_count = 0;
+  if (!it.ReadInt(&page_count))
     return scoped_ptr<AppListModel>();
-
   scoped_ptr<AppListModel> model(new AppListModel);
   model->SetSignedIn(signed_in);
-  for (int i = 0; i < app_count; ++i) {
-    scoped_ptr<AppListItemModel> item(UnpickleAppListItemModel(&it).Pass());
-    if (!item)
+  for (size_t p = 0; p < static_cast<size_t>(page_count); ++p) {
+    // Read the number of items in the page.
+    int item_count = 0;
+    if (!it.ReadInt(&item_count))
       return scoped_ptr<AppListModel>();
-    model->apps()->Add(item.release());
+    // Read each item in the page.
+    for (int i = 0; i < item_count; ++i) {
+      scoped_ptr<AppListItemModel> item(UnpickleAppListItemModel(&it).Pass());
+      if (!item)
+        return scoped_ptr<AppListModel>();
+      model->AddItemToPage(item.release(), p);
+    }
   }
 
   return model.Pass();
 }