Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(115)

Side by Side Diff: chrome/browser/bookmarks/bookmark_storage.cc

Issue 8828006: Makes all permanent nodes share the same node class so that visibility (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: More tweaks Created 9 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/bookmarks/bookmark_storage.h" 5 #include "chrome/browser/bookmarks/bookmark_storage.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/compiler_specific.h" 8 #include "base/compiler_specific.h"
9 #include "base/file_util.h" 9 #include "base/file_util.h"
10 #include "base/file_util_proxy.h" 10 #include "base/file_util_proxy.h"
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
82 BrowserThread::PostTask( 82 BrowserThread::PostTask(
83 BrowserThread::UI, FROM_HERE, 83 BrowserThread::UI, FROM_HERE,
84 base::Bind(&BookmarkStorage::OnLoadFinished, storage, 84 base::Bind(&BookmarkStorage::OnLoadFinished, storage,
85 bookmark_file_exists, path)); 85 bookmark_file_exists, path));
86 } 86 }
87 87
88 } // namespace 88 } // namespace
89 89
90 // BookmarkLoadDetails --------------------------------------------------------- 90 // BookmarkLoadDetails ---------------------------------------------------------
91 91
92 BookmarkLoadDetails::BookmarkLoadDetails(BookmarkNode* bb_node, 92 BookmarkLoadDetails::BookmarkLoadDetails(
93 BookmarkNode* other_folder_node, 93 BookmarkPermanentNode* bb_node,
94 BookmarkNode* mobile_folder_node, 94 BookmarkPermanentNode* other_folder_node,
95 BookmarkIndex* index, 95 BookmarkPermanentNode* mobile_folder_node,
96 int64 max_id) 96 BookmarkIndex* index,
97 int64 max_id)
97 : bb_node_(bb_node), 98 : bb_node_(bb_node),
98 other_folder_node_(other_folder_node), 99 other_folder_node_(other_folder_node),
99 mobile_folder_node_(mobile_folder_node), 100 mobile_folder_node_(mobile_folder_node),
100 index_(index), 101 index_(index),
101 max_id_(max_id), 102 max_id_(max_id),
102 ids_reassigned_(false) { 103 ids_reassigned_(false) {
103 } 104 }
104 105
105 BookmarkLoadDetails::~BookmarkLoadDetails() { 106 BookmarkLoadDetails::~BookmarkLoadDetails() {
106 } 107 }
(...skipping 130 matching lines...) Expand 10 before | Expand all | Expand 10 after
237 NOTREACHED(); 238 NOTREACHED();
238 return false; 239 return false;
239 } 240 }
240 241
241 std::string data; 242 std::string data;
242 if (!SerializeData(&data)) 243 if (!SerializeData(&data))
243 return false; 244 return false;
244 writer_.WriteNow(data); 245 writer_.WriteNow(data);
245 return true; 246 return true;
246 } 247 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698