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

Unified Diff: chrome/browser/chromeos/drive/drive_webapps_registry.cc

Issue 11190019: Fix 'for' construct which caused crash when VLOG is on. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 2 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chromeos/drive/drive_webapps_registry.cc
diff --git a/chrome/browser/chromeos/drive/drive_webapps_registry.cc b/chrome/browser/chromeos/drive/drive_webapps_registry.cc
index 2ba81add9e6e2603310ad4efeceb761cd90d5eb1..c9b3806bb17c482346f28665bdbbe421ba7f4a89 100644
--- a/chrome/browser/chromeos/drive/drive_webapps_registry.cc
+++ b/chrome/browser/chromeos/drive/drive_webapps_registry.cc
@@ -166,17 +166,15 @@ void DriveWebAppsRegistry::UpdateFromFeed(
if (VLOG_IS_ON(1)) {
std::vector<std::string> mime_types;
- for (size_t i = 0; app.primary_mimetypes().size(); ++i)
+ for (size_t i = 0; i < app.primary_mimetypes().size(); ++i)
mime_types.push_back(*app.primary_mimetypes()[i]);
- for (size_t i = 0; app.secondary_mimetypes().size(); ++i)
+ for (size_t i = 0; i < app.secondary_mimetypes().size(); ++i)
mime_types.push_back(*app.secondary_mimetypes()[i]);
-
std::vector<std::string> extensions;
- for (size_t i = 0; app.primary_extensions().size(); ++i)
+ for (size_t i = 0; i < app.primary_extensions().size(); ++i)
extensions.push_back(*app.primary_extensions()[i]);
- for (size_t i = 0; app.secondary_extensions().size(); ++i)
+ for (size_t i = 0; i < app.secondary_extensions().size(); ++i)
extensions.push_back(*app.secondary_extensions()[i]);
-
VLOG(1) << "Adding app " << app.app_name()
<< " to app registry. mime_types: "
<< JoinString(mime_types, ',') << " extensions: "
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698