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

Unified Diff: chrome/utility/media_galleries/picasa_albums_indexer.cc

Issue 18562007: Media Galleries API Picasa: Put INI indexing step into sandboxed utility process. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@0035-picasa-import-sandbox-pmp-reading
Patch Set: Created 7 years, 6 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 side-by-side diff with in-line comments
Download patch
Index: chrome/utility/media_galleries/picasa_albums_indexer.cc
diff --git a/chrome/utility/media_galleries/picasa_albums_indexer.cc b/chrome/utility/media_galleries/picasa_albums_indexer.cc
index a512e4e4833a49bd8b0c8114ac34a24d642d98c6..0181a29d134d44bc2597e0fb3326e2e628bcc528 100644
--- a/chrome/utility/media_galleries/picasa_albums_indexer.cc
+++ b/chrome/utility/media_galleries/picasa_albums_indexer.cc
@@ -2,6 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
+#include "chrome/common/media_galleries/picasa_types.h"
#include "chrome/utility/media_galleries/picasa_albums_indexer.h"
#include <vector>
@@ -43,7 +44,7 @@ class PicasaINIParser : public base::INIParser {
it != containing_albums.end(); ++it) {
AlbumImagesMap::iterator album_map_it = albums_images_->find(*it);
- // Ignore entry if the album UUID is not listed among in |album_uuids|
+ // Ignore entry if the album uid is not listed among in |album_uids|
// in the constructor. Happens if the PMP and INI files are inconsistent.
if (album_map_it == albums_images_->end())
continue;
@@ -59,11 +60,10 @@ class PicasaINIParser : public base::INIParser {
} // namespace
-PicasaAlbumsIndexer::PicasaAlbumsIndexer(
- const AlbumUUIDSet& album_uuids) {
- // Create an entry in the map for the valid album uuids.
- for (AlbumUUIDSet::const_iterator it = album_uuids.begin();
- it != album_uuids.end(); ++it) {
+PicasaAlbumsIndexer::PicasaAlbumsIndexer(const AlbumUIDSet& album_uids) {
+ // Create an entry in the map for the valid album uids.
+ for (AlbumUIDSet::const_iterator it = album_uids.begin();
+ it != album_uids.end(); ++it) {
albums_images_[*it] = AlbumImages();
}
}

Powered by Google App Engine
This is Rietveld 408576698