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

Issue 8143009: Reland 103263 - Revert 103263 - Merge 101221 - Reland 101111 - Only deliver extension messages to... (Closed)

Created:
9 years, 2 months ago by Aaron Boodman
Modified:
9 years, 2 months ago
Reviewers:
Aaron Boodman
CC:
chromium-reviews, Erik does not do reviews, Paweł Hajdan Jr., mihaip+watch_chromium.org, Aaron Boodman, darin-cc_chromium.org, brettw-cc_chromium.org
Visibility:
Public.

Description

Reland 103263 - Revert 103263 - Merge 101221 - Reland 101111 - Only deliver extension messages to contexts that care. ricow looked at the suspicious crashes and decided that they weren't new, and weren't introduced by this change, so back it goes. BUG=96544, 76571 Review URL: http://codereview.chromium.org/7888064 TBR=aa@chromium.org Review URL: http://codereview.chromium.org/8073023 TBR=aa@chromium.org Review URL: http://codereview.chromium.org/8103003 TBR=aa@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=104015

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+111 lines, -39 lines) Patch
M chrome/browser/debugger/extension_ports_remote_service.h View 1 chunk +3 lines, -2 lines 0 comments Download
M chrome/browser/debugger/extension_ports_remote_service.cc View 3 chunks +4 lines, -9 lines 0 comments Download
M chrome/browser/extensions/extension_message_service.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/extensions/extension_message_service.cc View 2 chunks +4 lines, -8 lines 0 comments Download
M chrome/browser/extensions/extension_messages_browsertest.cc View 4 chunks +3 lines, -10 lines 0 comments Download
M chrome/common/extensions/extension_messages.h View 1 chunk +5 lines, -0 lines 0 comments Download
M chrome/renderer/extensions/bindings_utils.h View 1 chunk +10 lines, -2 lines 0 comments Download
M chrome/renderer/extensions/bindings_utils.cc View 1 chunk +12 lines, -0 lines 0 comments Download
M chrome/renderer/extensions/event_bindings.cc View 1 chunk +1 line, -6 lines 0 comments Download
M chrome/renderer/extensions/extension_dispatcher.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/renderer/extensions/extension_dispatcher.cc View 2 chunks +8 lines, -0 lines 0 comments Download
M chrome/renderer/extensions/extension_helper.h View 1 chunk +2 lines, -0 lines 0 comments Download
M chrome/renderer/extensions/extension_helper.cc View 3 chunks +7 lines, -0 lines 0 comments Download
M chrome/renderer/extensions/renderer_extension_bindings.h View 2 chunks +6 lines, -1 line 0 comments Download
M chrome/renderer/extensions/renderer_extension_bindings.cc View 1 chunk +38 lines, -0 lines 0 comments Download
M chrome/renderer/resources/renderer_extension_bindings.js View 1 chunk +7 lines, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Aaron Boodman
9 years, 2 months ago (2011-10-04 23:04:41 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698