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

Issue 695203002: Remove an unnecessary #include section from render_process_host_impl.cc. (Closed)

Created:
6 years, 1 month ago by Sungmann Cho
Modified:
6 years, 1 month ago
CC:
chromium-reviews, darin-cc_chromium.org, nasko+codewatch_chromium.org, jam, creis+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Project:
chromium
Visibility:
Public.

Description

Remove an unnecessary #include section from render_process_host_impl.cc. #include <utility> in render_process_host_impl.cc is not necessary anymore. Committed: https://crrev.com/0a05c08f66d08329c835bef8430337774a9d0294 Cr-Commit-Position: refs/heads/master@{#302454}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -4 lines) Patch
M content/browser/renderer_host/render_process_host_impl.cc View 1 chunk +0 lines, -4 lines 0 comments Download

Messages

Total messages: 11 (2 generated)
Sungmann Cho
Please take a look. Thanks!
6 years, 1 month ago (2014-11-02 23:33:41 UTC) #2
Alexei Svitkine (slow)
lgtm
6 years, 1 month ago (2014-11-03 13:56:41 UTC) #3
Alexei Svitkine (slow)
not lgtm Could you fill out the contributor license agreement first, before submitting this? http://www.chromium.org/developers/contributing-code
6 years, 1 month ago (2014-11-03 13:59:37 UTC) #4
Alexei Svitkine (slow)
On 2014/11/03 13:59:37, Alexei Svitkine wrote: > not lgtm > > Could you fill out ...
6 years, 1 month ago (2014-11-03 15:26:11 UTC) #5
Sungmann Cho
On 2014/11/03 15:26:11, Alexei Svitkine wrote: > On 2014/11/03 13:59:37, Alexei Svitkine wrote: > > ...
6 years, 1 month ago (2014-11-03 17:09:53 UTC) #6
Alexei Svitkine (slow)
lgtm
6 years, 1 month ago (2014-11-03 17:24:15 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/695203002/1
6 years, 1 month ago (2014-11-03 17:34:34 UTC) #9
commit-bot: I haz the power
Committed patchset #1 (id:1)
6 years, 1 month ago (2014-11-03 18:52:46 UTC) #10
commit-bot: I haz the power
6 years, 1 month ago (2014-11-03 18:53:25 UTC) #11
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/0a05c08f66d08329c835bef8430337774a9d0294
Cr-Commit-Position: refs/heads/master@{#302454}

Powered by Google App Engine
This is Rietveld 408576698