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

Issue 3173020: FBTF: Remove unneeded #includes of <string> and <vector> from src/app and src/base. (Closed)

Created:
10 years, 4 months ago by viettrungluu
Modified:
9 years, 6 months ago
CC:
chromium-reviews, brettw-cc_chromium.org
Visibility:
Public.

Description

FBTF: Remove unneeded #includes of <string> and <vector> from src/app and src/base. BUG=none TEST=builds Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=56144

Patch Set 1 #

Patch Set 2 : argh #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -29 lines) Patch
M app/clipboard/scoped_clipboard_writer.h View 1 chunk +0 lines, -1 line 0 comments Download
M app/gtk_util.h View 1 chunk +0 lines, -1 line 0 comments Download
M app/os_exchange_data.h View 2 chunks +1 line, -2 lines 0 comments Download
M app/os_exchange_data_provider_gtk.h View 1 chunk +0 lines, -1 line 0 comments Download
M app/test_suite.h View 2 chunks +1 line, -3 lines 0 comments Download
M app/theme_provider.h View 2 chunks +1 line, -3 lines 0 comments Download
M base/debug_util.h View 2 chunks +1 line, -2 lines 0 comments Download
M base/event_recorder.h View 1 1 chunk +1 line, -1 line 0 comments Download
M base/i18n/file_util_icu.h View 2 chunks +1 line, -3 lines 0 comments Download
M base/i18n/number_formatting.h View 2 chunks +1 line, -3 lines 0 comments Download
M base/i18n/word_iterator.h View 2 chunks +1 line, -2 lines 0 comments Download
M base/observer_list_threadsafe.h View 1 chunk +0 lines, -1 line 0 comments Download
M base/resource_util.h View 2 chunks +3 lines, -3 lines 0 comments Download
M base/test/test_file_util.h View 2 chunks +1 line, -3 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
viettrungluu
10 years, 4 months ago (2010-08-14 07:17:17 UTC) #1
Miranda Callahan
10 years, 4 months ago (2010-08-14 17:42:38 UTC) #2
On 2010/08/14 07:17:17, viettrungluu wrote:
> 

LGTM!

Powered by Google App Engine
This is Rietveld 408576698