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

Issue 12296004: Make test_shell build on Win64 (Closed)

Created:
7 years, 10 months ago by jschuh
Modified:
7 years, 10 months ago
Reviewers:
jamesr
CC:
chromium-reviews, darin-cc_chromium.org
Visibility:
Public.

Description

Make test_shell build on Win64 Disable c4267 warning and fix a broken call site due to a deprecated function. BUG=166496 BUG=167187 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=183414

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -3 lines) Patch
M webkit/tools/test_shell/test_shell.gypi View 1 2 chunks +4 lines, -1 line 0 comments Download
M webkit/tools/test_shell/webwidget_host_win.cc View 1 1 chunk +1 line, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
jschuh
PTAL. Should just be a rubberstamp, but I don't want to TBR since it's not ...
7 years, 10 months ago (2013-02-17 16:51:52 UTC) #1
jamesr
no idea what this is lgtm
7 years, 10 months ago (2013-02-20 01:35:07 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jschuh@chromium.org/12296004/3002
7 years, 10 months ago (2013-02-20 01:58:35 UTC) #3
jschuh
7 years, 10 months ago (2013-02-20 05:03:56 UTC) #4
Message was sent while issue was closed.
Committed patchset #2 manually as r183414 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698