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

Issue 10821133: Move c/r/extensions/* into extensions namespace (Closed)

Created:
8 years, 4 months ago by Devlin
Modified:
8 years, 4 months ago
CC:
chromium-reviews, Aaron Boodman, mihaip-chromium-reviews_chromium.org, darin-cc_chromium.org, brettw-cc_chromium.org
Base URL:
http://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

Move c/r/extensions/* into extensions namespace Moved all classes and files in c/r/extensions into the extensions namespace. BUG=140056, 117261 TBR=jam@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=150402

Patch Set 1 : #

Patch Set 2 : Latest master for cq #

Unified diffs Side-by-side diffs Delta from patch set Stats (+420 lines, -2224 lines) Patch
M chrome/chrome_renderer.gypi View 3 chunks +6 lines, -6 lines 0 comments Download
M chrome/renderer/chrome_content_renderer_client.h View 4 chunks +6 lines, -3 lines 0 comments Download
M chrome/renderer/chrome_content_renderer_client.cc View 1 8 chunks +10 lines, -9 lines 0 comments Download
M chrome/renderer/chrome_render_view_observer.h View 1 3 chunks +6 lines, -3 lines 0 comments Download
M chrome/renderer/chrome_render_view_observer.cc View 1 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/renderer/extensions/api_definitions_natives.h View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/renderer/extensions/api_definitions_natives.cc View 1 3 chunks +7 lines, -11 lines 0 comments Download
M chrome/renderer/extensions/app_bindings.h View 2 chunks +6 lines, -2 lines 0 comments Download
M chrome/renderer/extensions/app_bindings.cc View 8 chunks +12 lines, -9 lines 0 comments Download
M chrome/renderer/extensions/app_window_custom_bindings.h View 1 chunk +2 lines, -3 lines 0 comments Download
M chrome/renderer/extensions/app_window_custom_bindings.cc View 2 chunks +3 lines, -4 lines 0 comments Download
M chrome/renderer/extensions/chrome_v8_context.h View 5 chunks +8 lines, -7 lines 0 comments Download
M chrome/renderer/extensions/chrome_v8_context.cc View 4 chunks +6 lines, -4 lines 0 comments Download
M chrome/renderer/extensions/chrome_v8_context_set.h View 3 chunks +5 lines, -1 line 0 comments Download
M chrome/renderer/extensions/chrome_v8_context_set.cc View 1 3 chunks +5 lines, -1 line 0 comments Download
M chrome/renderer/extensions/chrome_v8_context_set_unittest.cc View 3 chunks +6 lines, -2 lines 0 comments Download
M chrome/renderer/extensions/chrome_v8_extension.h View 3 chunks +8 lines, -7 lines 0 comments Download
M chrome/renderer/extensions/chrome_v8_extension.cc View 5 chunks +9 lines, -8 lines 0 comments Download
M chrome/renderer/extensions/chrome_v8_extension_handler.h View 2 chunks +3 lines, -0 lines 0 comments Download
M chrome/renderer/extensions/chrome_v8_extension_handler.cc View 2 chunks +4 lines, -0 lines 0 comments Download
A + chrome/renderer/extensions/dispatcher.h View 9 chunks +26 lines, -29 lines 0 comments Download
A + chrome/renderer/extensions/dispatcher.cc View 1 39 chunks +77 lines, -109 lines 0 comments Download
M chrome/renderer/extensions/event_bindings.h View 1 chunk +8 lines, -8 lines 0 comments Download
M chrome/renderer/extensions/event_bindings.cc View 9 chunks +17 lines, -19 lines 0 comments Download
M chrome/renderer/extensions/event_unittest.cc View 2 chunks +2 lines, -0 lines 0 comments Download
M chrome/renderer/extensions/extension_custom_bindings.h View 1 chunk +2 lines, -3 lines 0 comments Download
M chrome/renderer/extensions/extension_custom_bindings.cc View 2 chunks +3 lines, -4 lines 0 comments Download
D chrome/renderer/extensions/extension_dispatcher.h View 1 chunk +0 lines, -271 lines 0 comments Download
D chrome/renderer/extensions/extension_dispatcher.cc View 1 1 chunk +0 lines, -1169 lines 0 comments Download
M chrome/renderer/extensions/extension_groups.h View 2 chunks +3 lines, -0 lines 0 comments Download
M chrome/renderer/extensions/extension_helper.h View 5 chunks +7 lines, -4 lines 0 comments Download
M chrome/renderer/extensions/extension_helper.cc View 1 14 chunks +31 lines, -32 lines 0 comments Download
D chrome/renderer/extensions/extension_request_sender.h View 1 chunk +0 lines, -61 lines 0 comments Download
D chrome/renderer/extensions/extension_request_sender.cc View 1 chunk +0 lines, -167 lines 0 comments Download
D chrome/renderer/extensions/extension_resource_request_policy.h View 1 chunk +0 lines, -33 lines 0 comments Download
D chrome/renderer/extensions/extension_resource_request_policy.cc View 1 chunk +0 lines, -109 lines 0 comments Download
M chrome/renderer/extensions/json_schema_unittest.cc View 2 chunks +4 lines, -0 lines 0 comments Download
M chrome/renderer/extensions/miscellaneous_bindings.h View 3 chunks +3 lines, -3 lines 0 comments Download
M chrome/renderer/extensions/miscellaneous_bindings.cc View 4 chunks +6 lines, -6 lines 0 comments Download
M chrome/renderer/extensions/page_actions_custom_bindings.h View 1 chunk +2 lines, -3 lines 0 comments Download
M chrome/renderer/extensions/page_actions_custom_bindings.cc View 2 chunks +4 lines, -4 lines 0 comments Download
A + chrome/renderer/extensions/request_sender.h View 3 chunks +15 lines, -12 lines 0 comments Download
A + chrome/renderer/extensions/request_sender.cc View 8 chunks +24 lines, -25 lines 0 comments Download
A + chrome/renderer/extensions/resource_request_policy.h View 2 chunks +12 lines, -8 lines 0 comments Download
A + chrome/renderer/extensions/resource_request_policy.cc View 4 chunks +7 lines, -5 lines 0 comments Download
M chrome/renderer/extensions/runtime_custom_bindings.cc View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/renderer/extensions/send_request_natives.h View 2 chunks +4 lines, -6 lines 0 comments Download
M chrome/renderer/extensions/send_request_natives.cc View 1 chunk +4 lines, -6 lines 0 comments Download
M chrome/renderer/extensions/set_icon_natives.h View 1 chunk +4 lines, -6 lines 0 comments Download
M chrome/renderer/extensions/set_icon_natives.cc View 1 chunk +4 lines, -6 lines 0 comments Download
M chrome/renderer/extensions/tabs_custom_bindings.cc View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/renderer/extensions/tts_custom_bindings.cc View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/renderer/extensions/user_script_scheduler.h View 4 chunks +5 lines, -6 lines 0 comments Download
M chrome/renderer/extensions/user_script_scheduler.cc View 1 5 chunks +7 lines, -7 lines 0 comments Download
M chrome/renderer/extensions/user_script_slave.cc View 1 3 chunks +4 lines, -6 lines 0 comments Download
M chrome/renderer/extensions/webstore_bindings.h View 3 chunks +4 lines, -1 line 0 comments Download
M chrome/renderer/extensions/webstore_bindings.cc View 3 chunks +6 lines, -2 lines 0 comments Download
M chrome/renderer/resources/extensions/app_custom_bindings.js View 1 chunk +1 line, -1 line 0 comments Download
M chrome/renderer/resources/extensions/webstore_custom_bindings.js View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/base/chrome_render_view_test.h View 2 chunks +5 lines, -3 lines 0 comments Download
M chrome/test/base/chrome_render_view_test.cc View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Devlin
Refactor; please review when you get a chance. (Sorry for the large number of files.)
8 years, 4 months ago (2012-08-01 20:22:54 UTC) #1
miket_OOO
I knew this day would come! Thank you for doing this. For changed files, I ...
8 years, 4 months ago (2012-08-03 16:41:54 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rdevlin.cronin@chromium.org/10821133/8001
8 years, 4 months ago (2012-08-07 18:37:56 UTC) #3
commit-bot: I haz the power
Change committed as 150402
8 years, 4 months ago (2012-08-07 21:08:46 UTC) #4
not at google - send to devlin
8 years, 4 months ago (2012-08-08 07:33:47 UTC) #5
I think this change is going to make it hard to cherry pick anything that
touches chrome/extensions/renderer into the m22 branch...

Powered by Google App Engine
This is Rietveld 408576698