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

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

Created:
6 years, 8 months ago by aarya
Modified:
6 years, 8 months ago
Reviewers:
yurys
CC:
blink-reviews, sof, eae+blinkwatch, dglazkov+blink, adamk+blink_chromium.org, Inactive, rwlbuis
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=172584

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: 3 (0 generated)
aarya
6 years, 8 months ago (2014-04-25 04:09:05 UTC) #1
aarya
Committed patchset #1 manually as r172584.
6 years, 8 months ago (2014-04-25 04:09:24 UTC) #2
yurys
6 years, 8 months ago (2014-04-25 09:33:48 UTC) #3
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698