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

Issue 9595004: Coverity: Fix several pass-by-values. (Closed)

Created:
8 years, 9 months ago by James Hawkins
Modified:
8 years, 9 months ago
Reviewers:
groby-ooo-7-16
CC:
chromium-reviews, darin-cc_chromium.org
Visibility:
Public.

Description

Coverity: Fix several pass-by-values. CID_COUNT=6 CID=103346,103347,103348,103349,103354,103355 TEST=none BUG=none R=groby Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=125071

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -20 lines) Patch
M ppapi/proxy/ppb_video_capture_proxy.cc View 3 chunks +5 lines, -5 lines 0 comments Download
M ppapi/shared_impl/ppb_video_capture_shared.h View 2 chunks +9 lines, -7 lines 0 comments Download
M ppapi/shared_impl/ppb_video_capture_shared.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M ppapi/thunk/ppb_video_capture_api.h View 1 chunk +2 lines, -2 lines 0 comments Download
M webkit/plugins/ppapi/ppb_video_capture_impl.h View 1 chunk +2 lines, -2 lines 0 comments Download
M webkit/plugins/ppapi/ppb_video_capture_impl.cc View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
James Hawkins
8 years, 9 months ago (2012-03-04 02:31:03 UTC) #1
groby-ooo-7-16
8 years, 9 months ago (2012-03-05 19:32:17 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698