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

Issue 4710001: Split out command IDs from chrome_dll_resource.h. (Closed)

Created:
10 years, 1 month ago by Evan Martin
Modified:
9 years, 7 months ago
CC:
chromium-reviews, tim (not reviewing), ncarter (slow), John Grabowski, idana, ben+cc_chromium.org, Raghu Simha, Erik does not do reviews, brettw-cc_chromium.org, jam, Nirnimesh, Aaron Boodman, anantha, pam+watch_chromium.org, jshin+watch_chromium.org, Paweł Hajdan Jr., darin-cc_chromium.org, amit, davemoore+watch_chromium.org
Visibility:
Public.

Description

Split out command IDs from chrome_dll_resource.h. The name of this file is a leftover from Windows. Split the command ids (which are used on all platforms) out, leaving just the Windows-specific resource IDs in the original file. TEST=compiles Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=65453

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+408 lines, -413 lines) Patch
A chrome/app/chrome_command_ids.h View 1 chunk +275 lines, -0 lines 1 comment Download
M chrome/app/chrome_dll_resource.h View 2 chunks +0 lines, -289 lines 1 comment Download
M chrome/browser/app_controller_mac.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/app_controller_mac_unittest.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/autocomplete/autocomplete_edit.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/autocomplete/autocomplete_edit_view_browsertest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/autocomplete/autocomplete_edit_view_gtk.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/autocomplete/autocomplete_edit_view_win.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/automation/automation_provider.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/automation/automation_provider_observers.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/automation/testing_automation_provider.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/background_mode_manager.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/browser.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/browser_browsertest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/browser_commands_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/browser_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/character_encoding.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/chromeos/dom_ui/language_options_handler.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/chromeos/dom_ui/network_menu_ui.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/chromeos/dom_ui/wrench_menu_ui.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/chromeos/frame/browser_view.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/chromeos/status/feedback_menu_button.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/accelerators_cocoa.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/accelerators_cocoa.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/accelerators_cocoa_unittest.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/bookmarks/bookmark_menu_bridge_unittest.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/bookmarks/bookmark_menu_cocoa_controller.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/browser_window_cocoa.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/browser_window_controller.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/browser_window_controller.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/browser_window_controller_unittest.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/chrome_event_processing_window_unittest.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/encoding_menu_controller_delegate_mac.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/framed_browser_window_unittest.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/history_menu_bridge.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/history_menu_bridge_unittest.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/history_menu_cocoa_controller.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/history_menu_cocoa_controller_unittest.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/location_bar/autocomplete_text_field_editor.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/location_bar/autocomplete_text_field_editor_unittest.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/location_bar/location_bar_view_mac.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/location_bar/star_decoration.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/reload_button.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/reload_button_unittest.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/tab_strip_controller.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/tabpose_window.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/toolbar_controller.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/toolbar_controller_unittest.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/translate/translate_infobar_base.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/translate/translate_infobar_unittest.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/wrench_menu_button_cell_unittest.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/wrench_menu_controller.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/cocoa/wrench_menu_controller_unittest.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/collected_cookies_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/dom_ui/bookmarks_ui_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/dom_ui/new_tab_ui_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/dom_ui/options/options_ui_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/download/download_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/download/save_page_browsertest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/download/save_page_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/encoding_menu_controller.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/encoding_menu_controller_unittest.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/external_tab_container_win.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/global_keyboard_shortcuts_mac.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/global_keyboard_shortcuts_mac_unittest.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/gtk/accelerators_gtk.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/gtk/back_forward_button_gtk.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/gtk/browser_titlebar.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/gtk/browser_toolbar_gtk.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/gtk/browser_window_gtk.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/gtk/location_bar_view_gtk.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/gtk/menu_gtk.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/gtk/reload_button_gtk.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/gtk/tabs/tab_gtk.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/renderer_host/gtk_im_context_wrapper.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/repost_form_warning_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/sessions/session_restore_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ssl/ssl_browser_tests.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/sync/sync_ui_util_mac_unittest.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/tab_contents/render_view_context_menu.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/tab_contents/render_view_context_menu_gtk.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/tab_contents/render_view_context_menu_mac.mm View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/tab_contents/view_source_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/tab_restore_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/task_manager/task_manager_resource_providers.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/translate/options_menu_model.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/translate/translate_manager_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/views/about_ipc_dialog.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/views/accelerator_table_gtk.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/views/bookmark_bubble_view.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/views/constrained_window_win.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/views/find_bar_host_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/views/frame/browser_view.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/views/frame/glass_browser_frame_view.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/views/fullscreen_exit_bubble.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/views/location_bar/keyword_hint_view.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/views/location_bar/location_bar_view.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/views/location_bar/star_view.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/views/reload_button.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/views/tab_contents/render_view_context_menu_views.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/views/tab_icon_view.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/views/tabs/tab_dragging_test.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/views/task_manager_view.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/views/toolbar_view.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/views/wrench_menu.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/wrench_menu_model.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/wrench_menu_model_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/chrome_dll.gypi View 3 chunks +3 lines, -0 lines 0 comments Download
M chrome/chrome_exe.gypi View 1 chunk +1 line, -0 lines 1 comment Download
M chrome/chrome_tests.gypi View 4 chunks +4 lines, -0 lines 0 comments Download
M chrome/renderer/external_extension_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/automated_ui_tests/automated_ui_test_base.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/automated_ui_tests/automated_ui_test_test.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/automated_ui_tests/automated_ui_tests.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/automation/automation_messages_internal.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/automation/automation_proxy_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/automation/browser_proxy.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/interactive_ui/fast_shutdown_interactive_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/pyautolib/pyautolib.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/startup/feature_startup_test.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/tab_switching/tab_switching_test.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/ui/omnibox_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/ui/ui_test.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/webdriver/commands/create_session.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/webdriver/commands/session_with_id.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/webdriver/commands/webdriver_command.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/webdriver/session.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/worker/worker_uitest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome_frame/chrome_active_document.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 5 (0 generated)
Evan Martin
to make hte review simpler, note that all "1 chunk 14 lines" diffs are just ...
10 years, 1 month ago (2010-11-08 22:50:27 UTC) #1
Evan Martin
http://codereview.chromium.org/4710001/diff/1/3 File chrome/app/chrome_dll_resource.h (left): http://codereview.chromium.org/4710001/diff/1/3#oldcode260 chrome/app/chrome_dll_resource.h:260: #endif I deleted this block as well
10 years, 1 month ago (2010-11-08 22:50:54 UTC) #2
tony
LGTM. I just spot checked a few files. http://codereview.chromium.org/4710001/diff/1/111 File chrome/chrome_exe.gypi (right): http://codereview.chromium.org/4710001/diff/1/111#newcode203 chrome/chrome_exe.gypi:203: 'app/chrome_command_ids.h', ...
10 years, 1 month ago (2010-11-08 23:10:47 UTC) #3
Ben Goodger (Google)
http://codereview.chromium.org/4710001/diff/1/2 File chrome/app/chrome_command_ids.h (right): http://codereview.chromium.org/4710001/diff/1/2#newcode1 chrome/app/chrome_command_ids.h:1: // Copyright (c) 2010 The Chromium Authors. All rights ...
10 years, 1 month ago (2010-11-09 01:18:47 UTC) #4
evanm
10 years, 1 month ago (2010-11-09 01:28:07 UTC) #5
On 2010/11/09 01:18:47, Ben Goodger wrote:
> Would it make sense for this file to live in src/chrome/browser/ui now?

Yeah!  That is part of the reason I'm tackling this directory, it's filled with
all sorts of stuff that belongs elsewhere.  (In parallel I've been working on
moving some code from here into browser_main, allowing me to delete a bunch of
code in a place where it had been copied.)

I guess it's easy enough to rename now, though I note a bunch of other
directories (like workers for some reason) reach into it.  I guess those would
need to be fixed first.

Powered by Google App Engine
This is Rietveld 408576698