Chromium Code Reviews
[email protected] (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(399)

Issue 12283009: Rename "unload" to "suspend" for consistency (Closed)

Created:
7 years, 10 months ago by rpaquay
Modified:
7 years, 10 months ago
Reviewers:
Matt Perry, Chris Evans
CC:
chromium-reviews, Aaron Boodman, darin-cc_chromium.org, chromium-apps-reviews_chromium.org, asargent_no_longer_on_chrome
Base URL:
https://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

Rename "unload" to "suspend" for consistency Rename all constants, methods, etc. related to the "chrome.runtime.onSuspend" event to use the work "Suspend" instead of "Reload". Reload means a different thing in the rest of the codebase (it means "disabled" or about to be reloaded fresh from disk). BUG= [email protected] (for mechanical changes in chrome/.) Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=182945

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+81 lines, -81 lines) Patch
M chrome/browser/extensions/api/system_indicator/system_indicator_apitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/extensions/extension_context_menu_browsertest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/extensions/extension_process_manager.h View 2 chunks +8 lines, -8 lines 0 comments Download
M chrome/browser/extensions/extension_process_manager.cc View 7 chunks +25 lines, -25 lines 0 comments Download
M chrome/browser/extensions/lazy_background_page_apitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/extensions/platform_app_browsertest_util.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/renderer_host/chrome_render_message_filter.h View 1 chunk +3 lines, -3 lines 0 comments Download
M chrome/browser/renderer_host/chrome_render_message_filter.cc View 3 chunks +9 lines, -9 lines 0 comments Download
M chrome/common/chrome_switches.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/chrome_switches.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/extension_messages.h View 2 chunks +14 lines, -14 lines 0 comments Download
M chrome/renderer/extensions/dispatcher.h View 1 chunk +3 lines, -3 lines 0 comments Download
M chrome/renderer/extensions/dispatcher.cc View 3 chunks +13 lines, -13 lines 0 comments Download

Messages

Total messages: 12 (0 generated)
rpaquay
Matt, This refactoring is (somewhat) related to issue 165273. It seems we have a terminology ...
7 years, 10 months ago (2013-02-15 19:23:28 UTC) #1
Matt Perry
Good call, I agree. LGTM
7 years, 10 months ago (2013-02-15 19:26:06 UTC) #2
miket_OOO
Cool.
7 years, 10 months ago (2013-02-15 19:30:10 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/[email protected]/12283009/1
7 years, 10 months ago (2013-02-15 19:32:41 UTC) #4
commit-bot: I haz the power
Presubmit check for 12283009-1 failed and returned exit status 1. INFO:root:Found 13 file(s). Running presubmit ...
7 years, 10 months ago (2013-02-15 19:32:47 UTC) #5
rpaquay
Add cevans@ for review on chrome/common/extensions/extension_messages.h (trivial change)
7 years, 10 months ago (2013-02-15 20:24:22 UTC) #6
Chris Evans
On 2013/02/15 20:24:22, rpaquay wrote: > Add cevans@ for review on chrome/common/extensions/extension_messages.h (trivial > change) ...
7 years, 10 months ago (2013-02-15 20:49:36 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/[email protected]/12283009/1
7 years, 10 months ago (2013-02-15 21:41:14 UTC) #8
commit-bot: I haz the power
Presubmit check for 12283009-1 failed and returned exit status 1. INFO:root:Found 13 file(s). Running presubmit ...
7 years, 10 months ago (2013-02-15 21:41:24 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/[email protected]/12283009/1
7 years, 10 months ago (2013-02-15 21:51:52 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/[email protected]/12283009/1
7 years, 10 months ago (2013-02-15 22:29:57 UTC) #11
commit-bot: I haz the power
7 years, 10 months ago (2013-02-16 07:02:36 UTC) #12
Message was sent while issue was closed.
Change committed as 182945

Powered by Google App Engine
This is Rietveld 408576698