Chromium Code Reviews
Help | Chromium Project | Sign in
(195)

Issue 11299062: chrome: Update the remaining calls from RunAllPending() to RunUntilIdle(). (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 years, 6 months ago by tfarina
Modified:
2 years, 6 months ago
CC:
chromium-reviews, pam+watch_chromium.org, dhollowa+watch_chromium.org, rpetterson, dbeam+watch-autofill_chromium.org, chromium-apps-reviews_chromium.org, Aaron Boodman, darin-cc_chromium.org, dyu1, estade+watch_chromium.org, groby+spellwatch_chromium.org, Albert Bodenhamer, native-client-reviews_googlegroups.com, Ilya Sherman
Visibility:
Public.

Description

chrome: Update the remaining calls from RunAllPending() to RunUntilIdle(). RunAllPending() is deprecated and we should switch to RunUntilIdle(). BUG=131220 TBR=ben@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=168481

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+29 lines, -29 lines) Patch
M chrome/nacl/nacl_ipc_adapter_unittest.cc View 6 chunks +8 lines, -8 lines 0 comments Download
M chrome/renderer/autofill/password_autofill_manager_browsertest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/renderer/extensions/chrome_v8_context_set_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/renderer/safe_browsing/phishing_term_feature_extractor_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/renderer/spellchecker/spellcheck_unittest.cc View 8 chunks +8 lines, -8 lines 0 comments Download
M chrome/renderer/translate_helper_browsertest.cc View 6 chunks +6 lines, -6 lines 0 comments Download
M chrome/service/cloud_print/cloud_print_url_fetcher_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/base/testing_profile.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/tools/profiles/generate_profile.cc View 2 chunks +2 lines, -2 lines 0 comments Download
Commit: CQ not working?

Messages

Total messages: 10 (0 generated)
tfarina
TBRing...
2 years, 6 months ago (2012-11-18 13:24:17 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tfarina@chromium.org/11299062/1
2 years, 6 months ago (2012-11-18 13:24:30 UTC) #2
commit-bot: I haz the power
Sorry for I got bad news for ya. Compile failed with a clobber build on ...
2 years, 6 months ago (2012-11-18 13:30:35 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tfarina@chromium.org/11299062/1
2 years, 6 months ago (2012-11-18 13:46:17 UTC) #4
commit-bot: I haz the power
Sorry for I got bad news for ya. Compile failed with a clobber build on ...
2 years, 6 months ago (2012-11-18 13:49:46 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tfarina@chromium.org/11299062/1
2 years, 6 months ago (2012-11-18 16:18:53 UTC) #6
commit-bot: I haz the power
Sorry for I got bad news for ya. Compile failed with a clobber build on ...
2 years, 6 months ago (2012-11-18 16:21:08 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tfarina@chromium.org/11299062/1
2 years, 6 months ago (2012-11-18 23:46:23 UTC) #8
commit-bot: I haz the power
Sorry for I got bad news for ya. Compile failed with a clobber build on ...
2 years, 6 months ago (2012-11-18 23:49:48 UTC) #9
Ben Goodger (Google)
2 years, 6 months ago (2012-11-19 20:14:24 UTC) #10
lgtm
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld ec887be