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

Issue 1972001: graphics_WindowManagerGraphicsCapture test taking screenshots and comparing them to reference images (Closed)

Created:
10 years, 7 months ago by scheib
Modified:
9 years, 7 months ago
CC:
chromium-os-reviews_chromium.org, sosa+cc_chromium.org, seano, ericli, petkov+cc_chromium.org
Base URL:
ssh://git@chromiumos-git/autotest.git
Visibility:
Public.

Description

graphics_WindowManagerGraphicsCapture test taking screenshots and comparing them to reference images. BUG=2054 TEST=graphics_WindowManagerGraphicsCapture

Patch Set 1 #

Total comments: 1

Patch Set 2 : comment fixes #

Unified diffs Side-by-side diffs Delta from patch set Stats (+95 lines, -16 lines) Patch
M client/deps/glbench/src/main.h View 1 chunk +1 line, -0 lines 0 comments Download
M client/deps/glbench/src/windowmanagertest.cc View 1 2 chunks +62 lines, -12 lines 0 comments Download
M client/site_tests/graphics_WindowManagerGraphicsCapture/graphics_WindowManagerGraphicsCapture.py View 1 chunk +34 lines, -6 lines 0 comments Download
A client/site_tests/graphics_WindowManagerGraphicsCapture/screenshot1_reference View 0 chunks +-1 lines, --1 lines 0 comments Download
A client/site_tests/graphics_WindowManagerGraphicsCapture/screenshot2_reference View 0 chunks +-1 lines, --1 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
scheib
graphics_WindowManagerGraphicsCapture test now working, build system in place for imagemagick and perceptualdiff. Tested on two ...
10 years, 7 months ago (2010-05-05 01:09:37 UTC) #1
piman
LGTM with 1 comment http://codereview.chromium.org/1972001/diff/1/3 File client/deps/glbench/src/windowmanagertest.cc (right): http://codereview.chromium.org/1972001/diff/1/3#newcode68 client/deps/glbench/src/windowmanagertest.cc:68: // times a second. Comment ...
10 years, 7 months ago (2010-05-05 16:45:45 UTC) #2
Alexey Marinichev
10 years, 7 months ago (2010-05-05 17:56:43 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld 408576698