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

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

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 5 months ago by tfarina
Modified:
1 year, 5 months ago
CC:
chromium-reviews_chromium.org, 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) Lint Patch
M chrome/nacl/nacl_ipc_adapter_unittest.cc View 6 chunks +8 lines, -8 lines 0 comments 0 errors Download
M chrome/renderer/autofill/password_autofill_manager_browsertest.cc View 1 chunk +1 line, -1 line 0 comments 0 errors Download
M chrome/renderer/extensions/chrome_v8_context_set_unittest.cc View 1 chunk +1 line, -1 line 0 comments 0 errors Download
M chrome/renderer/safe_browsing/phishing_term_feature_extractor_unittest.cc View 1 chunk +1 line, -1 line 0 comments 0 errors Download
M chrome/renderer/spellchecker/spellcheck_unittest.cc View 8 chunks +8 lines, -8 lines 0 comments 0 errors Download
M chrome/renderer/translate_helper_browsertest.cc View 6 chunks +6 lines, -6 lines 0 comments 0 errors Download
M chrome/service/cloud_print/cloud_print_url_fetcher_unittest.cc View 1 chunk +1 line, -1 line 0 comments 0 errors Download
M chrome/test/base/testing_profile.cc View 1 chunk +1 line, -1 line 0 comments 0 errors Download
M chrome/tools/profiles/generate_profile.cc View 2 chunks +2 lines, -2 lines 0 comments 0 errors Download
Commit:

Messages

Total messages: 10
tfarina
TBRing...
1 year, 5 months ago #1
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tfarina@chromium.org/11299062/1
1 year, 5 months ago #2
I haz the power (commit-bot)
Sorry for I got bad news for ya. Compile failed with a clobber build on ...
1 year, 5 months ago #3
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tfarina@chromium.org/11299062/1
1 year, 5 months ago #4
I haz the power (commit-bot)
Sorry for I got bad news for ya. Compile failed with a clobber build on ...
1 year, 5 months ago #5
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tfarina@chromium.org/11299062/1
1 year, 5 months ago #6
I haz the power (commit-bot)
Sorry for I got bad news for ya. Compile failed with a clobber build on ...
1 year, 5 months ago #7
I haz the power (commit-bot)
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/tfarina@chromium.org/11299062/1
1 year, 5 months ago #8
I haz the power (commit-bot)
Sorry for I got bad news for ya. Compile failed with a clobber build on ...
1 year, 5 months ago #9
Ben Goodger (Google)
1 year, 5 months ago #10
lgtm
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6