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

Issue 9662050: Coverity: Fix a few uninitialized vars. (Closed)

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

Description

Coverity: Fix a few uninitialized vars. CID_COUNT=3 CID=103471,103509,103510 BUG=none TEST=none R=groby Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=126211

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
M content/browser/renderer_host/render_sandbox_host_linux.cc View 1 chunk +1 line, -1 line 0 comments Download
M ipc/ipc_channel_reader.cc View 1 chunk +1 line, -0 lines 0 comments Download
M webkit/plugins/ppapi/event_conversion.cc View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
James Hawkins
8 years, 9 months ago (2012-03-12 01:45:13 UTC) #1
groby-ooo-7-16
lgtm
8 years, 9 months ago (2012-03-12 16:57:41 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jhawkins@chromium.org/9662050/1
8 years, 9 months ago (2012-03-12 17:59:35 UTC) #3
commit-bot: I haz the power
8 years, 9 months ago (2012-03-12 17:59:37 UTC) #4
Presubmit check for 9662050-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    content/browser
    webkit/plugins/ppapi

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

Powered by Google App Engine
This is Rietveld 408576698