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

Side by Side Diff: chrome/browser/chromeos/drive/change_list_loader.h

Issue 14271007: Remove public bool ChangeListLoader::loaded(). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 8 months 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) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #ifndef CHROME_BROWSER_CHROMEOS_DRIVE_CHANGE_LIST_LOADER_H_ 5 #ifndef CHROME_BROWSER_CHROMEOS_DRIVE_CHANGE_LIST_LOADER_H_
6 #define CHROME_BROWSER_CHROMEOS_DRIVE_CHANGE_LIST_LOADER_H_ 6 #define CHROME_BROWSER_CHROMEOS_DRIVE_CHANGE_LIST_LOADER_H_
7 7
8 #include <map> 8 #include <map>
9 #include <string> 9 #include <string>
10 #include <utility> 10 #include <utility>
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
42 public: 42 public:
43 ChangeListLoader(DriveResourceMetadata* resource_metadata, 43 ChangeListLoader(DriveResourceMetadata* resource_metadata,
44 DriveScheduler* scheduler, 44 DriveScheduler* scheduler,
45 DriveWebAppsRegistry* webapps_registry); 45 DriveWebAppsRegistry* webapps_registry);
46 ~ChangeListLoader(); 46 ~ChangeListLoader();
47 47
48 // Adds and removes the observer. 48 // Adds and removes the observer.
49 void AddObserver(ChangeListLoaderObserver* observer); 49 void AddObserver(ChangeListLoaderObserver* observer);
50 void RemoveObserver(ChangeListLoaderObserver* observer); 50 void RemoveObserver(ChangeListLoaderObserver* observer);
51 51
52 bool loaded() const { return loaded_; }
53
54 // Starts the change list loading first from the cache. If loading from the 52 // Starts the change list loading first from the cache. If loading from the
55 // cache is successful, runs |callback| and starts loading from the server 53 // cache is successful, runs |callback| and starts loading from the server
56 // if needed (i.e. the cache is old). If loading from the cache is 54 // if needed (i.e. the cache is old). If loading from the cache is
57 // unsuccessful, starts loading from the server, and runs |callback| to 55 // unsuccessful, starts loading from the server, and runs |callback| to
58 // tell the result to the caller. 56 // tell the result to the caller.
59 // 57 //
60 // If |directory_fetch_info| is not empty, the directory will be fetched 58 // If |directory_fetch_info| is not empty, the directory will be fetched
61 // first from the server, so the UI can show the directory contents 59 // first from the server, so the UI can show the directory contents
62 // instantly before the entire change list loading is complete. 60 // instantly before the entire change list loading is complete.
63 // 61 //
(...skipping 220 matching lines...) Expand 10 before | Expand all | Expand 10 after
284 282
285 // Note: This should remain the last member so it'll be destroyed and 283 // Note: This should remain the last member so it'll be destroyed and
286 // invalidate its weak pointers before any other members are destroyed. 284 // invalidate its weak pointers before any other members are destroyed.
287 base::WeakPtrFactory<ChangeListLoader> weak_ptr_factory_; 285 base::WeakPtrFactory<ChangeListLoader> weak_ptr_factory_;
288 DISALLOW_COPY_AND_ASSIGN(ChangeListLoader); 286 DISALLOW_COPY_AND_ASSIGN(ChangeListLoader);
289 }; 287 };
290 288
291 } // namespace drive 289 } // namespace drive
292 290
293 #endif // CHROME_BROWSER_CHROMEOS_DRIVE_CHANGE_LIST_LOADER_H_ 291 #endif // CHROME_BROWSER_CHROMEOS_DRIVE_CHANGE_LIST_LOADER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698