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

Issue 10830021: Coverity: Initialize member variables. (Closed)

Created:
8 years, 5 months ago by James Hawkins
Modified:
8 years, 5 months ago
CC:
chromium-reviews, tfarina, darin-cc_chromium.org, dcheng
Visibility:
Public.

Description

Coverity: Initialize member variables. CID_COUNT=7 CID=13432,14429,16922,101542,101734,102304,102305 BUG=none TEST=none R=tbreisacher TBR=jam@chromium.org,tommi@chromium.org,ben@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=148592

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -5 lines) Patch
M courgette/disassembler_elf_32_x86.cc View 1 chunk +7 lines, -1 line 0 comments Download
M courgette/patcher_x86_32.h View 1 chunk +3 lines, -1 line 0 comments Download
M ppapi/native_client/src/trusted/plugin/file_downloader.h View 1 chunk +2 lines, -0 lines 0 comments Download
M ui/base/dragdrop/os_exchange_data_provider_win.cc View 1 chunk +1 line, -1 line 0 comments Download
M ui/gfx/render_text_win.cc View 1 chunk +4 lines, -1 line 0 comments Download
M ui/views/controls/native_control_win.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/plugins/npapi/webplugin_ime_win.cc View 2 chunks +2 lines, -0 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
James Hawkins
8 years, 5 months ago (2012-07-25 21:26:13 UTC) #1
Tyler Breisacher (Chromium)
lgtm
8 years, 5 months ago (2012-07-25 21:26:59 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/10830021/1
8 years, 5 months ago (2012-07-25 21:27:38 UTC) #3
commit-bot: I haz the power
Try job failure for 10830021-1 (retry) on mac_rel for step "compile" (clobber build). It's a ...
8 years, 5 months ago (2012-07-25 21:53:13 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jhawkins@chromium.org/10830021/1
8 years, 5 months ago (2012-07-25 21:56:46 UTC) #5
commit-bot: I haz the power
Try job failure for 10830021-1 (retry) on linux_rel for step "browser_tests". It's a second try, ...
8 years, 5 months ago (2012-07-26 03:06:16 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jhawkins@chromium.org/10830021/1
8 years, 5 months ago (2012-07-26 17:24:49 UTC) #7
commit-bot: I haz the power
8 years, 5 months ago (2012-07-26 18:34:26 UTC) #8
Change committed as 148592

Powered by Google App Engine
This is Rietveld 408576698