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

Issue 119463003: Rename ContentBrowserClient::RenderProcessHostCreated to RenderProcessWillLaunch. It is called each… (Closed)

Created:
7 years ago by Avi (use Gerrit)
Modified:
7 years ago
Reviewers:
jam
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, android-webview-reviews_chromium.org, jochen+watch_chromium.org
Visibility:
Public.

Description

Rename ContentBrowserClient::RenderProcessHostCreated to RenderProcessWillLaunch. It is called each time a render process will be launched, and possibly several times during the lifetime of a RenderProcessHost. BUG=none TEST=no change TBR=ben@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=242037

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -9 lines) Patch
M android_webview/browser/aw_content_browser_client.h View 1 chunk +1 line, -1 line 0 comments Download
M android_webview/browser/aw_content_browser_client.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/chrome_content_browser_client.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/chrome_content_browser_client.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/renderer_host/render_process_host_impl.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/public/browser/content_browser_client.h View 1 chunk +2 lines, -2 lines 0 comments Download
M content/shell/browser/shell_content_browser_client.h View 1 chunk +1 line, -1 line 0 comments Download
M content/shell/browser/shell_content_browser_client.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
Avi (use Gerrit)
The complete RenderProcessHostObserver interface will have the following callbacks for render process lifecycle events: RenderProcessWillLaunch ...
7 years ago (2013-12-19 21:59:37 UTC) #1
jam
lgtm
7 years ago (2013-12-20 00:50:58 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/119463003/1
7 years ago (2013-12-20 01:32:26 UTC) #3
commit-bot: I haz the power
7 years ago (2013-12-20 07:53:34 UTC) #4
Message was sent while issue was closed.
Change committed as 242037

Powered by Google App Engine
This is Rietveld 408576698