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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 3080031: Delete obviously unneeded forward declarations in chrome/browser/*.h. (Closed)

Created:
10 years, 4 months ago by viettrungluu
Modified:
9 years, 6 months ago
CC:
chromium-reviews, John Grabowski, pam+watch_chromium.org, ben+cc_chromium.org, Paweł Hajdan Jr.
Visibility:
Public.

Description

Delete obviously unneeded forward declarations in chrome/browser/*.h. BUG=none TEST=builds Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=55037

Patch Set 1 #

Patch Set 2 : hrm #

Patch Set 3 : bah #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -53 lines) Patch
M chrome/browser/app_controller_mac.h View 2 chunks +1 line, -2 lines 0 comments Download
M chrome/browser/app_modal_dialog.h View 2 chunks +1 line, -4 lines 0 comments Download
M chrome/browser/browser.h View 1 chunk +0 lines, -3 lines 0 comments Download
M chrome/browser/browser_accessibility_manager_win.h View 1 chunk +0 lines, -2 lines 0 comments Download
M chrome/browser/browser_init.h View 1 chunk +0 lines, -6 lines 0 comments Download
M chrome/browser/browser_main_win.h View 2 chunks +1 line, -2 lines 0 comments Download
M chrome/browser/browser_process.h View 1 chunk +0 lines, -2 lines 0 comments Download
M chrome/browser/browser_theme_pack.h View 2 chunks +1 line, -2 lines 0 comments Download
M chrome/browser/browser_window.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/configuration_policy_provider.h View 1 chunk +0 lines, -2 lines 0 comments Download
M chrome/browser/encoding_menu_controller.h View 2 chunks +1 line, -2 lines 0 comments Download
M chrome/browser/external_protocol_handler.h View 2 chunks +1 line, -2 lines 0 comments Download
M chrome/browser/file_watcher.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/find_bar.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/gpu_process_host.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/js_modal_dialog.h View 1 chunk +4 lines, -0 lines 0 comments Download
M chrome/browser/jsmessage_box_client.h View 2 chunks +1 line, -2 lines 0 comments Download
M chrome/browser/memory_purger.h View 2 chunks +1 line, -2 lines 0 comments Download
M chrome/browser/remove_rows_table_model.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/repost_form_warning_controller.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/tab_menu_model.h View 1 chunk +0 lines, -2 lines 0 comments Download
M chrome/browser/task_manager.h View 2 chunks +1 line, -4 lines 0 comments Download
M chrome/browser/toolbar_model.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/utility_process_host.h View 2 chunks +1 line, -3 lines 0 comments Download
M chrome/browser/visitedlink_master.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/test/testing_browser_process.h View 1 chunk +0 lines, -4 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
viettrungluu
10 years, 4 months ago (2010-08-05 05:48:55 UTC) #1
Lei Zhang
10 years, 4 months ago (2010-08-05 06:34:14 UTC) #2 
LGTM if it passes the trybots.

Powered by Google App Engine
This is Rietveld 408576698