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

Issue 8686014: Remove child_process.h includes from chrome (specifically WaitForDebugger). I moved the browser p... (Closed)

Created:
9 years ago by jam
Modified:
9 years ago
Reviewers:
Dirk Pranke
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, native-client-reviews_googlegroups.com, jam, dpranke-watch+content_chromium.org
Visibility:
Public.

Description

Remove child_process.h includes from chrome (specifically WaitForDebugger). I moved the browser process usage to content. For the NaCl usage, it wasn't working anyways since it wasn't being passed to the process (and if it was, the message box wouldn't work on Windows because of sandbox). When this is needed again, we can figure out a good way of doing it. BUG=98716 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=112259

Patch Set 1 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -32 lines) Patch
M chrome/browser/DEPS View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/chrome_browser_main.cc View 2 chunks +0 lines, -8 lines 0 comments Download
M chrome/common/chrome_switches.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/common/chrome_switches.cc View 1 chunk +0 lines, -3 lines 0 comments Download
M chrome/nacl/nacl_main.cc View 1 chunk +0 lines, -19 lines 0 comments Download
M content/browser/browser_main.cc View 2 chunks +8 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
jam
9 years ago (2011-11-30 01:29:50 UTC) #1
Dirk Pranke
9 years ago (2011-11-30 02:11:17 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698