Coverity PASS_BY_VALUE fix.

Coverity PASS_BY_VALUE fix.

CID=104112, 104110, 104109
BUG=None
TEST=
R=jeremya@chromium.org

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

[jeremya, 2012-06-01 06:05:01]

lgtm

[commit-bot: I haz the power, 2012-06-01 06:06:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/scr@chromium.org/10443075/1

[commit-bot: I haz the power, 2012-06-01 06:06:17]

Presubmit check for 10443075-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit Messages **
If this change requires manual test instructions to QA team, add
TEST=[instructions].

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome/browser/ui/gtk
    chrome/browser/ui
    chrome/browser/ui/extensions
    chrome/browser/ui/cocoa

Presubmit checks took 2.0s to calculate.

Was the presubmit check useful? Please send feedback & hate mail to
maruel@chromium.org!

[Mihai Parparita -not on Chrome, 2012-06-01 06:31:05]

LGTM for chrome/browser/ui/extensions

[commit-bot: I haz the power, 2012-06-01 06:43:42]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/scr@chromium.org/10443075/1

[commit-bot: I haz the power, 2012-06-01 06:43:49]

Presubmit check for 10443075-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit Messages **
If this change requires manual test instructions to QA team, add
TEST=[instructions].

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome/browser/ui/gtk
    chrome/browser/ui
    chrome/browser/ui/cocoa

Presubmit checks took 1.5s to calculate.

[Sheridan Rawlins, 2012-06-01 18:53:18]

Scott, does your OWNER @ chrome/browser/ui count for the three missing OWNER
LG's?  Let me know when you review chrome/browser/ui so I can add others if
necessary.

Thanks,

-SCR

[sky, 2012-06-12 20:59:04]

LGTM

[commit-bot: I haz the power, 2012-06-12 21:11:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/scr@chromium.org/10443075/1

[commit-bot: I haz the power, 2012-06-12 21:11:58]

Failed to apply patch for
chrome/browser/ui/views/extensions/shell_window_views.cc:
While running patch -p1 --forward --force;
patching file chrome/browser/ui/views/extensions/shell_window_views.cc
Hunk #1 FAILED at 101.
Hunk #2 succeeded at 444 (offset 95 lines).
1 out of 2 hunks FAILED -- saving rejects to file
chrome/browser/ui/views/extensions/shell_window_views.cc.rej

[commit-bot: I haz the power, 2012-06-12 21:52:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/scr@chromium.org/10443075/9001

[commit-bot: I haz the power, 2012-06-12 23:33:45]

Change committed as 141785