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

Issue 10876029: [Android] Upstreaming process number increase (Closed)

Created:
8 years, 4 months ago by gone
Modified:
8 years, 3 months ago
Reviewers:
Satish
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, klobag.chromium
Base URL:
http://git.chromium.org/chromium/src.git@master
Visibility:
Public.

Description

[Android] Upstreaming process number increase Increases the number of processes we allow from 5 to 6. BUG=143821 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=155343

Patch Set 1 #

Patch Set 2 : Rebasing #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -3 lines) Patch
A + content/public/android/java/src/org/chromium/content/app/SandboxedProcessService5.java View 1 chunk +1 line, -1 line 0 comments Download
M content/public/android/java/src/org/chromium/content/browser/AndroidBrowserProcess.java View 1 chunk +8 lines, -1 line 0 comments Download
M content/public/android/java/src/org/chromium/content/browser/SandboxedProcessLauncher.java View 1 1 chunk +1 line, -1 line 0 comments Download
M content/shell/android/java/AndroidManifest.xml View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
gone
8 years, 4 months ago (2012-08-22 23:43:10 UTC) #1
Satish
I posted a comment in the linked bug and would prefer we don't land this ...
8 years, 4 months ago (2012-08-23 09:51:40 UTC) #2
Yaron
So can we land this? It sounds like Satish's concerns have been addressed in the ...
8 years, 3 months ago (2012-09-05 18:25:53 UTC) #3
Satish
lgtm
8 years, 3 months ago (2012-09-06 08:42:04 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dfalcantara@chromium.org/10876029/7001
8 years, 3 months ago (2012-09-06 18:26:56 UTC) #5
commit-bot: I haz the power
8 years, 3 months ago (2012-09-07 06:30:30 UTC) #6
Change committed as 155343

Powered by Google App Engine
This is Rietveld 408576698