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

Unified Diff: chrome/browser/extensions/extension_warning_set.cc

Issue 9433044: Fix extensions compile issues for Android on sync_unit_tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Tracked issue for default_height in a bug and put bug # here. Created 8 years, 10 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/browser/extensions/extension_warning_set.cc
diff --git a/chrome/browser/extensions/extension_warning_set.cc b/chrome/browser/extensions/extension_warning_set.cc
index 84ca4a0e26fa75efdb7a24f0cb582f2b90d850da..77437437aa4b92cf276888cd1d7506f5a97c8846 100644
--- a/chrome/browser/extensions/extension_warning_set.cc
+++ b/chrome/browser/extensions/extension_warning_set.cc
@@ -111,7 +111,7 @@ void ExtensionWarningSet::SetWarning(ExtensionWarningSet::WarningType type,
void ExtensionWarningSet::ClearWarnings(
const std::set<ExtensionWarningSet::WarningType>& types) {
bool deleted_anything = false;
- for (const_iterator i = warnings_.begin(); i != warnings_.end();) {
+ for (iterator i = warnings_.begin(); i != warnings_.end();) {
asargent_no_longer_on_chrome 2012/02/22 21:47:55 Out of curiousity, are you switching to using a no
David Trainor- moved to gerrit 2012/02/22 21:51:42 It looks like our Android stl port of std::set doe
if (types.find(i->warning_type()) != types.end()) {
deleted_anything = true;
warnings_.erase(i++);

Powered by Google App Engine
This is Rietveld 408576698