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

Issue 9350006: Revert "Unrevert 119770 - "Event pages: remember events that the page registered for," (Closed)

Created:
8 years, 10 months ago by Matt Perry
Modified:
8 years, 10 months ago
Reviewers:
Aaron Boodman
CC:
chromium-reviews, Aaron Boodman, darin-cc_chromium.org, mihaip+watch_chromium.org, brettw-cc_chromium.org
Visibility:
Public.

Description

Revert "Unrevert 119770 - "Event pages: remember events that the page registered for," Original CL: Add support for broadcasted events. Add an extension.onInstalled event that is run when the event page first starts." Fixed ASAN bug with previous incarnation. Reason for revert: Some WebstoreInlineInstallTest tests have started failing. Might be this CL. TBR=aa BUG=81752, 112391 TEST=no Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=120678

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+89 lines, -294 lines) Patch
M chrome/browser/extensions/extension_event_router.h View 3 chunks +3 lines, -27 lines 0 comments Download
M chrome/browser/extensions/extension_event_router.cc View 14 chunks +47 lines, -86 lines 0 comments Download
M chrome/browser/extensions/extension_host.cc View 2 chunks +4 lines, -7 lines 0 comments Download
M chrome/browser/extensions/extension_process_manager.h View 1 chunk +0 lines, -4 lines 0 comments Download
M chrome/browser/extensions/extension_process_manager.cc View 3 chunks +3 lines, -20 lines 0 comments Download
M chrome/browser/extensions/lazy_background_page_apitest.cc View 5 chunks +27 lines, -62 lines 0 comments Download
M chrome/browser/renderer_host/chrome_render_message_filter.h View 1 chunk +0 lines, -4 lines 0 comments Download
M chrome/browser/renderer_host/chrome_render_message_filter.cc View 3 chunks +0 lines, -20 lines 0 comments Download
M chrome/common/common_resources.grd View 1 chunk +0 lines, -1 line 0 comments Download
D chrome/common/extensions/api/experimental.extension.json View 1 chunk +0 lines, -15 lines 0 comments Download
M chrome/common/extensions/api/extension_api.cc View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/common/extensions/docs/experimental.html View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/common/extensions/docs/js/api_page_generator.js View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/common/extensions/extension_messages.h View 2 chunks +1 line, -13 lines 0 comments Download
M chrome/renderer/extensions/event_bindings.cc View 3 chunks +1 line, -12 lines 0 comments Download
M chrome/test/data/extensions/api_test/lazy_background_page/broadcast_event/background.js View 1 chunk +3 lines, -2 lines 0 comments Download
D chrome/test/data/extensions/api_test/lazy_background_page/on_installed/background.js View 1 chunk +0 lines, -7 lines 0 comments Download
D chrome/test/data/extensions/api_test/lazy_background_page/on_installed/manifest.json View 1 chunk +0 lines, -11 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Matt Perry
8 years, 10 months ago (2012-02-07 01:28:39 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698