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

Issue 267573002: Merge 171240 "Fix compilation on Chromium OS after r171165" (Closed)

Created:
6 years, 7 months ago by aarya
Modified:
6 years, 7 months ago
Reviewers:
yurys
CC:
blink-reviews, dglazkov+blink, sof, eae+blinkwatch, adamk+blink_chromium.org, Inactive
Visibility:
Public.

Description

Merge 171240 "Fix compilation on Chromium OS after r171165" > Fix compilation on Chromium OS after r171165 > > r171165 broke compilation on Chromium OS bot and we had to revert Blink roll. > ... > ../../../../../../../home/chrome-bot/chrome_root/src/third_party/WebKit/Source/core/dom/CharacterData.cpp:146:36: > error: 'realCount' may be used uninitialized in this function > ... > I simply added initialization for realCount variables to make the compiler happy. > > [1] http://build.chromium.org/p/chromium.chromiumos/builders/ChromiumOS%20%28x86%29/builds/20955 > > BUG=349898 > TBR=sigbjornf@opera.com > > Review URL: https://codereview.chromium.org/232243004 TBR=yurys@chromium.org Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=173029

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M Source/core/dom/CharacterData.cpp View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
aarya
6 years, 7 months ago (2014-04-30 21:17:04 UTC) #1
aarya
6 years, 7 months ago (2014-04-30 21:17:11 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r173029.

Powered by Google App Engine
This is Rietveld 408576698