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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 8491036: Cleanup: Remove unneeded forward declarations from various chrome subdirectories. (Closed)

Created:
9 years, 1 month ago by Lei Zhang
Modified:
9 years, 1 month ago
Reviewers:
Paweł Hajdan Jr.
CC:
chromium-reviews, Erik does not do reviews, kkania, mihaip+watch_chromium.org, Aaron Boodman, Paweł Hajdan Jr., jshin+watch_chromium.org
Visibility:
Public.

Description

Cleanup: Remove unneeded forward declarations from various chrome subdirectories. BUG=none TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=109951

Patch Set 1 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -53 lines) Patch
M chrome/common/extensions/extension_l10n_util.h View 2 chunks +1 line, -3 lines 0 comments Download
M chrome/common/extensions/extension_permission_set.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/common/json_schema_validator.h View 2 chunks +1 line, -2 lines 0 comments Download
M chrome/common/logging_chrome.h View 2 chunks +1 line, -5 lines 0 comments Download
M chrome/default_plugin/plugin_impl_gtk.h View 2 chunks +1 line, -4 lines 0 comments Download
M chrome/default_plugin/plugin_impl_mac.h View 2 chunks +1 line, -4 lines 0 comments Download
M chrome/service/cloud_print/cloud_print_helpers.h View 2 chunks +3 lines, -5 lines 0 comments Download
M chrome/service/cloud_print/cloud_print_proxy_backend.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/service/cloud_print/print_system.h View 2 chunks +1 line, -2 lines 0 comments Download
M chrome/service/service_utility_process_host.h View 1 chunk +0 lines, -5 lines 0 comments Download
M chrome/test/base/testing_browser_process.h View 1 chunk +0 lines, -4 lines 0 comments Download
M chrome/test/base/testing_profile.h View 3 chunks +0 lines, -6 lines 0 comments Download
M chrome/test/base/ui_test_utils.h View 1 chunk +0 lines, -3 lines 0 comments Download
M chrome/test/ui/ui_test.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/test/webdriver/commands/mouse_commands.h View 1 chunk +0 lines, -5 lines 0 comments Download
M chrome/test/webdriver/commands/webdriver_command.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/test/webdriver/webdriver_session.h View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
Lei Zhang
9 years, 1 month ago (2011-11-11 10:11:45 UTC) #1
Paweł Hajdan Jr.
LGTM
9 years, 1 month ago (2011-11-14 12:05:10 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thestig@chromium.org/8491036/6001
9 years, 1 month ago (2011-11-14 20:02:35 UTC) #3
commit-bot: I haz the power
9 years, 1 month ago (2011-11-14 21:03:52 UTC) #4 
Change committed as 109951

Powered by Google App Engine
This is Rietveld 408576698