Revert 224987 "Fix crash when showing error in wrench menu"
commit7a30564f865b87ca9b70c23ae14ace578b8cb026
authorxiyuan@chromium.org <xiyuan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>
Tue, 24 Sep 2013 16:45:24 +0000 (24 16:45 +0000)
committerxiyuan@chromium.org <xiyuan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>
Tue, 24 Sep 2013 16:45:24 +0000 (24 16:45 +0000)
treeb48906d34aceec3a9ba60cb250f0b4ebf107eb57
parent724e0ccfe7866c77be5579d0a682507e307b2e98
Revert 224987 "Fix crash when showing error in wrench menu"

Android unit_test failures are real:
http://build.chromium.org/p/chromium.linux/builders/Android%20Tests/builds/8139/steps/unit_tests/logs/stdio#failure1

[----------] 3 tests from ProfileSyncServiceAutofillTest
[ RUN      ] ProfileSyncServiceAutofillTest.HasProfileEmptySync
[FATAL:global_error_service.cc(26)] Check failed: error.
[FATAL:global_error_service.cc(26)] Check failed: error.
[ CRASHED      ]

> Fix crash when showing error in wrench menu
>
> This is speculative fix for a crash in
> WrenchMenuModel::AddGlobalErrorMenuItems(). My guess is that we're
> modifying a list while iterating over it. Fix is to move the modifying
> call out of the loop.
>
> BUG=278543
> TEST=Can't repro the crash. Just built and ran Chrome.
> NOTRY=true
>
> Review URL: https://chromiumcodereview.appspot.com/23435011

TBR=sail@chromium.org

Review URL: https://codereview.chromium.org/23439011

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225009 0039d316-1c4b-4281-b951-d872f2087c98
chrome/browser/ui/global_error/global_error_service.cc
chrome/browser/ui/toolbar/wrench_menu_model.cc