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

Issue 8617001: A few less TOUCH_UIs. (Closed)

Created:
9 years, 1 month ago by jonathan.backer
Modified:
9 years, 1 month ago
Reviewers:
apatrick_chromium
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, apatrick_chromium, dpranke-watch+content_chromium.org
Visibility:
Public.

Description

A few less TOUCH_UIs. There is no more TOUCH_UI. This is a NO-OP because defined(UI_COMPOSITOR_IMAGE_TRANSPORT) ==> !defined(TOOLKIT_USES_GTK). BUG=None TEST=None Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=111142

Patch Set 1 : "" #

Patch Set 2 : Ooops 's/&&/||' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -16 lines) Patch
M content/browser/gpu/gpu_process_host.h View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/gpu/gpu_process_host.cc View 5 chunks +7 lines, -7 lines 0 comments Download
M content/browser/gpu/gpu_process_host_ui_shim.h View 1 1 chunk +1 line, -2 lines 0 comments Download
M content/browser/gpu/gpu_process_host_ui_shim.cc View 1 3 chunks +3 lines, -3 lines 0 comments Download
M content/common/gpu/gpu_channel_manager.cc View 1 chunk +1 line, -1 line 0 comments Download
M gpu/command_buffer/service/gles2_cmd_decoder.cc View 1 1 chunk +2 lines, -1 line 0 comments Download
M gpu/gpu.gyp View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
jonathan.backer
9 years, 1 month ago (2011-11-21 18:27:01 UTC) #1
apatrick_chromium
LGTM
9 years, 1 month ago (2011-11-21 20:07:22 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/backer@chromium.org/8617001/5001
9 years, 1 month ago (2011-11-21 22:39:57 UTC) #3
commit-bot: I haz the power
9 years, 1 month ago (2011-11-22 00:08:55 UTC) #4
Try job failure for 8617001-5001 (retry) on linux_rel for steps "browser_tests,
ui_tests".
It's a second try, previously, steps "browser_tests, ui_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

Powered by Google App Engine
This is Rietveld 408576698