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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 16614005: Use a direct include of strings headers in content/child/. (Closed)

Created:
7 years, 6 months ago by Avi (use Gerrit)
Modified:
7 years, 6 months ago
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam
Visibility:
Public.

Description

Use a direct include of strings headers in content/child/. BUG=247723 TEST=none TBR=ben@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=205418

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -9 lines) Patch
M content/child/child_process.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/child/child_thread.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/child/indexed_db/indexed_db_dispatcher.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/child/np_channel_base.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/child/npobject_util.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/child/resource_dispatcher.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/child/socket_stream_dispatcher.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/child/web_database_observer_impl.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/child/webmessageportchannel_impl.h View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/avi@chromium.org/16614005/1
7 years, 6 months ago (2013-06-10 20:43:58 UTC) #1
commit-bot: I haz the power
Step "update" is always a major failure. Look at the try server FAQ for more ...
7 years, 6 months ago (2013-06-11 00:11:24 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/avi@chromium.org/16614005/1
7 years, 6 months ago (2013-06-11 00:18:43 UTC) #3
commit-bot: I haz the power
Step "update" is always a major failure. Look at the try server FAQ for more ...
7 years, 6 months ago (2013-06-11 00:20:24 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/avi@chromium.org/16614005/1
7 years, 6 months ago (2013-06-11 00:22:48 UTC) #5
commit-bot: I haz the power
7 years, 6 months ago (2013-06-11 04:58:08 UTC) #6
Message was sent while issue was closed. 
Change committed as 205418

Powered by Google App Engine
This is Rietveld 408576698