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

Issue 60393005: Revert 232631 - Revert 232499 "Move ManifestHandler to top-level extensions." (Closed)

Created:
7 years, 1 month ago by Yoyo Zhou
Modified:
7 years, 1 month ago
Reviewers:
benwells
CC:
chromium-reviews, extensions-reviews_chromium.org, plundblad+watch_chromium.org, aboxhall+watch_chromium.org, yoshiki+watch_chromium.org, yuzo+watch_chromium.org, rginda+watch_chromium.org, dmazzoni+watch_chromium.org, dtseng+watch_chromium.org, chromium-apps-reviews_chromium.org
Visibility:
Public.

Description

Revert 232631 - Revert 232499 "Move ManifestHandler to top-level extensions." Was not the cause of breakage. See bug for that. BUG=314916 R=benwells@chromium.org TBR=benwells@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=233308

Patch Set 1 #

Patch Set 2 : rebase #

Patch Set 3 : deps #

Unified diffs Side-by-side diffs Delta from patch set Stats (+54 lines, -678 lines) Patch
M chrome/browser/extensions/api/identity/experimental_identity_api.cc View 1 2 1 chunk +1 line, -1 line 0 comments Download
M chrome/chrome_common.gypi View 1 chunk +0 lines, -2 lines 0 comments Download
M chrome/chrome_tests_unit.gypi View 1 2 chunks +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/commands/commands_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/extension_action/browser_action_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/extension_action/page_action_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/extension_action/script_badge_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/file_browser_handlers/file_browser_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/file_handlers/file_handlers_parser.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/i18n/default_locale_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/identity/oauth2_manifest_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/input_ime/input_components_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/managed_mode_private/managed_mode_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/media_galleries_private/media_galleries_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/omnibox/omnibox_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/plugins/plugins_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/sockets/sockets_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/speech/tts_engine_manifest_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/spellcheck/spellcheck_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/storage/storage_schema_manifest_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/system_indicator/system_indicator_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/url_handlers/url_handlers_parser.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/background_info.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/csp_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/extension.cc View 2 chunks +1 line, -1 line 0 comments Download
M chrome/common/extensions/extension_file_util.cc View 2 chunks +1 line, -1 line 0 comments Download
M chrome/common/extensions/incognito_handler.h View 1 chunk +1 line, -1 line 0 comments Download
D chrome/common/extensions/manifest_handler.h View 1 chunk +0 lines, -138 lines 0 comments Download
D chrome/common/extensions/manifest_handler.cc View 1 chunk +0 lines, -205 lines 0 comments Download
D chrome/common/extensions/manifest_handler_unittest.cc View 1 1 chunk +0 lines, -281 lines 0 comments Download
M chrome/common/extensions/manifest_handlers/app_isolation_info.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/manifest_handlers/app_launch_info.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/manifest_handlers/content_scripts_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/manifest_handlers/externally_connectable.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/manifest_handlers/icons_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/manifest_handlers/kiosk_mode_info.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/manifest_handlers/minimum_chrome_version_checker.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/manifest_handlers/nacl_modules_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/manifest_handlers/offline_enabled_info.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/manifest_handlers/requirements_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/manifest_handlers/sandboxed_page_info.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/manifest_handlers/settings_overrides_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/manifest_handlers/shared_module_info.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/manifest_handlers/theme_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/manifest_url_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/mime_types_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/web_accessible_resources_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/webview_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M extensions/DEPS View 1 2 1 chunk +3 lines, -3 lines 0 comments Download
A + extensions/common/manifest_handler.h View 2 chunks +3 lines, -3 lines 0 comments Download
A + extensions/common/manifest_handler.cc View 1 chunk +1 line, -1 line 0 comments Download
A + extensions/common/manifest_handler_unittest.cc View 1 1 chunk +1 line, -1 line 0 comments Download
M extensions/extensions.gyp View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
Yoyo Zhou
benwells: this is TBR, since it's basically unchanged since the first time it landed.
7 years, 1 month ago (2013-11-05 23:48:56 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/yoz@chromium.org/60393005/1
7 years, 1 month ago (2013-11-05 23:56:10 UTC) #2
commit-bot: I haz the power
Failed to apply patch for extensions/common/manifest_handler_unittest.cc: While running patch -p1 --forward --force --no-backup-if-mismatch; A extensions/common/manifest_handler_unittest.cc ...
7 years, 1 month ago (2013-11-05 23:56:29 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/yoz@chromium.org/60393005/60001
7 years, 1 month ago (2013-11-06 01:35:15 UTC) #4
benwells
On 2013/11/06 01:35:15, I haz the power (commit-bot) wrote: > CQ is trying da patch. ...
7 years, 1 month ago (2013-11-06 06:04:38 UTC) #5
commit-bot: I haz the power
Retried try job too often on linux_aura for step(s) browser_tests http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_aura&number=94523
7 years, 1 month ago (2013-11-06 06:18:52 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/yoz@chromium.org/60393005/60001
7 years, 1 month ago (2013-11-06 07:19:36 UTC) #7
commit-bot: I haz the power
Retried try job too often on linux_aura for step(s) browser_tests http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_aura&number=94712
7 years, 1 month ago (2013-11-06 10:47:50 UTC) #8
Yoyo Zhou
7 years, 1 month ago (2013-11-06 18:42:43 UTC) #9
Message was sent while issue was closed.
Committed patchset #3 manually as r233308 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698