Cleanup: Remove base::environment_vector and base::file_handle_mapping_vector to StudlyCaps.

Cleanup: Remove base::environment_vector and base::file_handle_mapping_vector to StudlyCaps.

BUG=none
TEST=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=123438

[Evan Martin, 2012-02-22 07:04:11]

base lgtm, do you want me to look at the rest? you probably need to find owners

https://chromiumcodereview.appspot.com/9429039/diff/1/base/process_util_posix.cc
File base/process_util_posix.cc (right):

https://chromiumcodereview.appspot.com/9429039/diff/1/base/process_util_posix...
base/process_util_posix.cc:431: j = changes.begin(); j != changes.end(); j++) {
What weird style (the j on the next line)!
Not your code, I know.

[Lei Zhang, 2012-02-22 07:16:37]

It's all either cookie cutter change or fixing lint errors. If you don't mind,
feel free to look at the rest.

jam: OWNERS for content/browser

https://chromiumcodereview.appspot.com/9429039/diff/1/base/process_util_posix.cc
File base/process_util_posix.cc (right):

https://chromiumcodereview.appspot.com/9429039/diff/1/base/process_util_posix...
base/process_util_posix.cc:431: j = changes.begin(); j != changes.end(); j++) {
On 2012/02/22 07:04:11, Evan Martin wrote:
> What weird style (the j on the next line)!
> Not your code, I know.

Done.

[jam, 2012-02-22 16:12:10]

content lgtm

this always bothered me, thanks for cleaning it up!

[Lei Zhang, 2012-02-22 19:36:00]

willchan: OWNERS for base and net
abodenha: OWNERS for chrome/browser/printing/cloud_print

[Albert Bodenhamer, 2012-02-22 19:37:36]

lgtm
Reviewed cloud print code only.

[Evan Martin, 2012-02-22 20:41:38]

LGTM for whole thing

[willchan no longer on Chromium, 2012-02-22 23:42:51]

lgtm

[commit-bot: I haz the power, 2012-02-22 23:43:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/thestig@chromium.org/9429039/5006

[commit-bot: I haz the power, 2012-02-22 23:43:41]

Can't apply patch for file content/browser/ppapi_plugin_process_host.cc.
While running patch -p0 --forward --force;
patching file content/browser/ppapi_plugin_process_host.cc
Hunk #1 FAILED at 1.
Hunk #4 succeeded at 189 (offset 1 line).
1 out of 4 hunks FAILED -- saving rejects to file
content/browser/ppapi_plugin_process_host.cc.rej

[commit-bot: I haz the power, 2012-02-23 01:54:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/thestig@chromium.org/9429039/11001

[commit-bot: I haz the power, 2012-02-23 02:02:46]

Try job failure for 9429039-11001 (retry) on android for steps "build, Compile".
It's a second try, previously, steps "build, Compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android&nu...

[commit-bot: I haz the power, 2012-02-24 02:58:24]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/thestig@chromium.org/9429039/16015

[commit-bot: I haz the power, 2012-02-24 04:40:22]

Change committed as 123438