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

Issue 11773027: Enable the multi-process host on Windows by default. (Closed)

Created:
7 years, 11 months ago by alexeypa (please no reviews)
Modified:
7 years, 11 months ago
Reviewers:
Sergey Ulanov, Wez
CC:
chromium-reviews, jamiewalch+watch_chromium.org, dcaiafa+watch_chromium.org, simonmorris+watch_chromium.org, hclam+watch_chromium.org, wez+watch_chromium.org, amit, sanjeevr, garykac+watch_chromium.org, lambroslambrou+watch_chromium.org, rmsousa+watch_chromium.org, alexeypa+watch_chromium.org, sergeyu+watch_chromium.org
Visibility:
Public.

Description

Enable the multi-process host on Windows by default. BUG=134694 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=175584

Patch Set 1 #

Total comments: 4
Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -1 line) Patch
M remoting/remoting.gyp View 1 chunk +12 lines, -1 line 4 comments Download

Messages

Total messages: 8 (0 generated)
alexeypa (please no reviews)
PTAL.
7 years, 11 months ago (2013-01-07 17:59:18 UTC) #1
Sergey Ulanov
https://chromiumcodereview.appspot.com/11773027/diff/1/remoting/remoting.gyp File remoting/remoting.gyp (right): https://chromiumcodereview.appspot.com/11773027/diff/1/remoting/remoting.gyp#newcode11 remoting/remoting.gyp:11: 'variables': { Variables inside variables? I think it's better ...
7 years, 11 months ago (2013-01-07 20:45:53 UTC) #2
alexeypa (please no reviews)
https://chromiumcodereview.appspot.com/11773027/diff/1/remoting/remoting.gyp File remoting/remoting.gyp (right): https://chromiumcodereview.appspot.com/11773027/diff/1/remoting/remoting.gyp#newcode11 remoting/remoting.gyp:11: 'variables': { On 2013/01/07 20:45:53, sergeyu wrote: > Variables ...
7 years, 11 months ago (2013-01-07 21:02:30 UTC) #3
Sergey Ulanov
LGTM. Yay on finally getting it enabled by default! https://chromiumcodereview.appspot.com/11773027/diff/1/remoting/remoting.gyp File remoting/remoting.gyp (right): https://chromiumcodereview.appspot.com/11773027/diff/1/remoting/remoting.gyp#newcode11 remoting/remoting.gyp:11: ...
7 years, 11 months ago (2013-01-08 00:29:10 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/alexeypa@chromium.org/11773027/1
7 years, 11 months ago (2013-01-08 01:17:31 UTC) #5
Wez
lgtm
7 years, 11 months ago (2013-01-08 01:23:31 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/alexeypa@chromium.org/11773027/1
7 years, 11 months ago (2013-01-08 17:30:40 UTC) #7
commit-bot: I haz the power
7 years, 11 months ago (2013-01-08 21:56:20 UTC) #8
Message was sent while issue was closed.
Change committed as 175584

Powered by Google App Engine
This is Rietveld 408576698