History log of /external/autotest/client/cros/multimedia/display_facade_native.py
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
0309dc99e4f2ffae6211c80e4e46a62579807486 03-Feb-2017 Achuith Bhandarkar <achuith@chromium.org> autotest: Replace WaitForJavaScriptExpression with WaitForJavaScriptCondition.

BUG=chromium:682812
TEST=trybot

Change-Id: I8a71fe24c04f31d401a1925251c5d221accb4d46
Reviewed-on: https://chromium-review.googlesource.com/436528
Commit-Ready: Achuith Bhandarkar <achuith@chromium.org>
Tested-by: Achuith Bhandarkar <achuith@chromium.org>
Reviewed-by: Jacob Dufault <jdufault@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
eb5db992b43406802e561e7940866185eb996073 24-Jan-2017 Achuith Bhandarkar <achuith@chromium.org> autotest: Use explicit timeout keyword in JavaScript calls.

BUG=chromium:682812
TEST=trybots

Change-Id: I1ae37ef0625d29d299878698b9cda8468eb430bd
Reviewed-on: https://chromium-review.googlesource.com/431959
Commit-Ready: Achuith Bhandarkar <achuith@chromium.org>
Tested-by: Achuith Bhandarkar <achuith@chromium.org>
Reviewed-by: Jacob Dufault <jdufault@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
4309a2d9e224ada1f4257136b142a715a648ea6a 27-Sep-2016 Wai-Hong Tam <waihong@google.com> multimedia: Support getting and setting screen orientation

Get/set the screen orientation via using telemetry to open a display
setting page and changing the setting.

BUG=b:31608632
TEST=pending

Change-Id: I9bca22b37b8d92a55c02bcb16abbb14cf962b857
Reviewed-on: https://chromium-review.googlesource.com/390495
Commit-Ready: Wai-Hong Tam <waihong@google.com>
Tested-by: Wai-Hong Tam <waihong@google.com>
Reviewed-by: Cheng-Yi Chiang <cychiang@chromium.org>
Reviewed-by: Ben Cheng <bccheng@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
f0280df2cd2d90b7f2ae3635e6571afd966904d9 30-Jun-2016 Tom Wai-Hong Tam <waihong@chromium.org> Chameleon: Stabilize the screen before starting measurement

Before starting measurement, it does:
* hide the typing cursor in the url bar;
* wait the pop-up window faded-out.

BUG=b:28885370
TEST=Ran the existing cheets_HBOGOFpsTest.

Change-Id: I66554ce9607dd44830105f5f0158138a27d5f52e
Reviewed-on: https://chromium-review.googlesource.com/357964
Commit-Ready: Wai-Hong Tam <waihong@chromium.org>
Tested-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Cheng-Yi Chiang <cychiang@chromium.org>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
72fdd4f355caa93e1925f7a77a18a9168f2416b7 02-Mar-2016 Tom Wai-Hong Tam <waihong@chromium.org> Chameleon: Increase the retry number and timeout value for mirroring

Have to increase these values for the dongle connection, e.g.
Link needs timeout value at least 3;
One Spring needs the retry number 4 but another is fine to use 2.

Change these values to larger to reduce noise.

BUG=chromium:582489
TEST=Ran the display_SwitchMode.fast_switch test on Link and Spring.

Change-Id: I874b58f3f160021bc51dd2201998741e19aa24fa
Reviewed-on: https://chromium-review.googlesource.com/329946
Commit-Ready: Wai-Hong Tam <waihong@chromium.org>
Tested-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
d752da7dc11e9f11c4a38fa14ad90493c612671a 24-Feb-2016 Tom Wai-Hong Tam <waihong@chromium.org> Chameleon: Remove the hard-coded wait time for checking the mode

It fixes a TODO. Poll the mode status instead of a hard-coded wait.

Doing some experiment on some slow devices, e.g. parrot, daisy, pi,
big, and jerry. The retry process is needed, using 2 passes most of
cases, as sending keys is flaky. Using 1 second for the wait_for_value
timeout is enough.

BUG=None
TEST=Ran the display_SwitchMode test.

Change-Id: Ic441d2650441561deee38ff0f5f297a7e577f9aa
Reviewed-on: https://chromium-review.googlesource.com/329223
Commit-Ready: Wai-Hong Tam <waihong@chromium.org>
Tested-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
74ef809f5647db614c5b8215303cc8f2f96c7d8a 22-Dec-2015 Moja Hsu <mojahsu@google.com> [autotest] facade_resource: Use tab descriptor to store tab object

We rely on tab[-1] to close or get the last opened tab. It may have
problem if there is a bug in telemetry if the last opened tab is not put
on tab[-1]. Now we ues tab descriptor to choose tab explicitly.

BUG=chromium:547801
TEST=Ran the tests which use load_url and close_tab:
display_ResolutionList.extended,
display_ResolutionList.mirrored,
display_Tearing.extended,
display_Tearing.mirrored,
audio_MediaBasicVerification.wav
passed.

Change-Id: I0cfd62fb1e6ebd27bfd858224df36f49fe7a6a85
Reviewed-on: https://chromium-review.googlesource.com/319663
Commit-Ready: Hsu Wei-Cheng <mojahsu@chromium.org>
Tested-by: Hsu Wei-Cheng <mojahsu@chromium.org>
Reviewed-by: Wai-Hong Tam <waihong@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
fa4a4125a948b8c9f60dbf1c6aa2f4d9e6880732 20-Nov-2015 Tom Wai-Hong Tam <waihong@chromium.org> Chameleon: Retry the flaky method get_display_modes()

It is about the issue crbug.com/547801. Some experiment got the result that
the first call of get_available_resolutions() or get_display_modes() always
failed. Add a retry logic to work around it. Should investigate the root
cause later.

BUG=chromium:547801
TEST=Ran the test display_ResolutionList.extended passed.

Change-Id: Ie6739f1c1b422152c4f467553b780ed676fecbc1
Reviewed-on: https://chromium-review.googlesource.com/313651
Commit-Ready: Wai-Hong Tam <waihong@chromium.org>
Tested-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
f9cafd96a07eadf92d85cb82d7538e9a2706b1c1 29-Oct-2015 Cheng-Yi Chiang <cychiang@chromium.org> [autotest] multimedia: Rename display test extension

Rename display test extension to multimedia test extension.
This extension will be used for both display/video/audio tests.

BUG=chromium:534953
TEST=run display_ClientChameleonConnection test

Change-Id: I527c8930dc1b16083a24783d506acd66132b2cef
Reviewed-on: https://chromium-review.googlesource.com/309549
Commit-Ready: Cheng-Yi Chiang <cychiang@chromium.org>
Tested-by: Cheng-Yi Chiang <cychiang@chromium.org>
Reviewed-by: Cheng-Yi Chiang <cychiang@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
766444e7b94e8e15e91de574a654351e0c9fc5b5 22-Oct-2015 Cheng-Yi Chiang <cychiang@chromium.org> [autotest] browser_facade_native: Add a facade to access browser

This facade is dedicated to access browser and use facade_resource to
access Chrome to open adn close tab.
Rename close_tab to close_tab_by_index in facade_resource.

BUG=chromium:546490
TEST=run display_ResolutionList.extended test.

Change-Id: I3788ecabd4a7d31d1481ba260b40bb67a91a35fc
Reviewed-on: https://chromium-review.googlesource.com/308081
Commit-Ready: Cheng-Yi Chiang <cychiang@chromium.org>
Tested-by: Cheng-Yi Chiang <cychiang@chromium.org>
Reviewed-by: Cheng-Yi Chiang <cychiang@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
04dd632f3fd5bf6575b58dc988f0066195fad552 28-Oct-2015 Cheng-Yi Chiang <cychiang@chromium.org> [autotest] multimedia: Add facade_resource module to access Chrome

Centralizes the methods to access Chrome extension and tabs to
facade_resource module. It will provdes methods to different facades
including display facade, audio facade, and upcoming browser facade.
Removes the unused local audio facade to simplify the refactoring.

BUG=chromium:546490
TEST=run audio_AudioBasicHeadphone, display_Tearing.mirrored,
display_ResolutionList.extended, display_Resolution.mirrored test on
link.

Change-Id: Ida68cecb3414be32ccb7863ced9b9949972538ad
Reviewed-on: https://chromium-review.googlesource.com/309461
Commit-Ready: Cheng-Yi Chiang <cychiang@chromium.org>
Tested-by: Cheng-Yi Chiang <cychiang@chromium.org>
Reviewed-by: Cheng-Yi Chiang <cychiang@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
4c3c0607c047129cce10d6b0d52a816e3e61fd07 30-Sep-2015 Yuli Huang <yuli@chromium.org> Set fullscreen by chrome.windows.update() instead of F11 key.

BUG=chromium:534963
Test=run display display_SwitchMode test on daisy

Change-Id: I5644c02169e92c82f09fddb64b001a5f38a3f2d9
Reviewed-on: https://chromium-review.googlesource.com/303263
Commit-Ready: Yuli Huang <yuli@chromium.org>
Tested-by: Yuli Huang <yuli@chromium.org>
Reviewed-by: Mussa Kiroga <mussa@chromium.org>
Reviewed-by: Hung-ying Tyan <tyanh@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
5455a73bea22963cbbb34fac3cc037df3ba482eb 10-Sep-2015 Chin-Huang Lin <chinlin@google.com> Chameleon: Fix return value of move_to_display().

Under current implementation of move_to_display(), if it returns
True, it doesn't mean that the window is successfully moved to
the indicated screen, but just means that the actions are done.
This will confuse the user since they can't detect whether it
really succeed without taking a look at the screen.

This patch fixes this problem by adding a check process which
really checks whether the result window is as expected.

BUG=chromium:530308
TEST=Ran display_Resolution.extended on nyan_blaze and it passed.
Ran display_ResolutionList.extended on nyan_blaze with issue
529829 unhandled, under which move_to_display() should fail,
and move_to_display() returned False as expected.

Change-Id: I62c02d58ecac618166505df477ee66863af7760a
Reviewed-on: https://chromium-review.googlesource.com/299040
Commit-Ready: Ting-Yuan Cheng <tingyuan@chromium.org>
Tested-by: Chin-Huang Lin <chinlin@google.com>
Reviewed-by: Yuli Huang <yuli@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
7acadb8f9046c908fe3d34b72dff75c1d34497b4 10-Sep-2015 Chin-Huang Lin <chinlin@google.com> Chameleon: Retry to load extensions after failure.

Currently if get_display_info(), get_window_info() or move_to_display()
fails to load the extension, which may happen due to delay, it will
directly give up rather than try again. Though they are all wrapped
with _retry_chrome_call decorator, it seems to be useless under current
implementation.

This patch tries to fix this problem by re-raising KeyError (which will
be raised when the browser fails to get the extension) as chrome.Error,
and so that such errors will be caught by _retry_chrome_call.

BUG=chromium:528795
TEST=Ran display_Resolution.extended on nyan_blaze under normal condition
and it passed.
Unable to reproduce KeyError issue. Need more observation on the
future test results.

Change-Id: Id6a99b71a9cf91036fbab3d0cded0a55e22d6bfd
Reviewed-on: https://chromium-review.googlesource.com/298990
Commit-Ready: Chin-Huang Lin <chinlin@google.com>
Tested-by: Chin-Huang Lin <chinlin@google.com>
Reviewed-by: Yuli Huang <yuli@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
09f9d5aa21078c54b9e0003154361608acfcddfc 05-Sep-2015 Chin-Huang Lin <chinlin@google.com> Chameleon: Add display_Tearing test

This test checks whether there is screen tearing issue.
It switches between pure color frames, and check whether some frame is
teared or missed/redundant/wrong.

BUG=chromium:528667
TEST=Ran tests on peppy, nyan_big and both passed.

Change-Id: I65886cfa5c26a5c68a239a1a315e32686d9db85a
Reviewed-on: https://chromium-review.googlesource.com/290671
Commit-Ready: Chin-Huang Lin <chinlin@google.com>
Tested-by: Chin-Huang Lin <chinlin@google.com>
Reviewed-by: Yuli Huang <yuli@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
10a1c6a6773d2f42ee27e396ecf318b7bac457b3 10-Sep-2015 Chin-Huang Lin <chinlin@google.com> Chameleon: Disable fullscreen before closing a tab.

Currently there is a bug in Chrome.browser.tabs.Close(),
(see https://code.google.com/p/chromium/issues/detail?id=530219)
which will cancel fullscreen mode if current state is fullscreen
without changing system configuration.

I've found this bug in a previous CL, but I handled this bug only
in chameleon_screen_test. This patch moves the parts to close_tab()
and adds some comments to make it clearer and more general.
(see https://chromium-review.googlesource.cddom/#/c/295661/)

BUG=None
TEST=Ran display_Resolution on nyan_blaze and it passed.

Change-Id: If97437a3db0ab57b0da7f4af9ef853a7fd2250f5
Reviewed-on: https://chromium-review.googlesource.com/298763
Commit-Ready: Chin-Huang Lin <chinlin@google.com>
Tested-by: Chin-Huang Lin <chinlin@google.com>
Reviewed-by: Chin-Huang Lin <chinlin@google.com>
/external/autotest/client/cros/multimedia/display_facade_native.py
c02fefede41df9f1375b4b67d5f631b715e91ac1 26-Aug-2015 Chin-Huang Lin <chinlin@google.com> Chameleon: Use Telemetry ChromeOS Autotest Extension
for Chameleon display tests.

Since autotest_ext has provided access to chrome.windows object,
we can simplify multimedia_test_extension. At the same time,
we can also more easily develop new API functions, including
set_full_screen and move_to_display.

Action done:
1. Simplify multimedia_test_extension, and rename it to
display_test_extension since audio tests do not use
this extension currently and so that the new name is
more explicit.
2. Add move_to_display() and set_fullscreen() into
display_facade.
3. Modify
chameleon_screen_test.py
video_GlitchDetection.py
display_ClientChameleonConnection.py
to prevent regression.

BUG=chromium:526916
Test=Tested on peppy with display_Resolution and passed.

Change-Id: I18dda14bfbb23b27993b1c3d63785e22176eefb7
Reviewed-on: https://chromium-review.googlesource.com/295661
Commit-Ready: Chin-Huang Lin <chinlin@google.com>
Tested-by: Chin-Huang Lin <chinlin@google.com>
Reviewed-by: Yuli Huang <yuli@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
e0b08e6170b57f90262726eb7f04e059cb47419c 11-Aug-2015 Hsinyu Chao <hychao@chromium.org> Revert "autotest: move constants to common_lib"

This reverts commit a4ac7a8f67d61f8a616724f9426dbc0fce71ce69.

BUG=chromium:519353

Change-Id: Iec2873ad22b5bb90e46a9e5b323f9626a2c07d9b
Reviewed-on: https://chromium-review.googlesource.com/292347
Reviewed-by: Hsinyu Chao <hychao@chromium.org>
Commit-Queue: Hsinyu Chao <hychao@chromium.org>
Tested-by: Hsinyu Chao <hychao@chromium.org>
Reviewed-by: Sonny Rao <sonnyrao@chromium.org>
Commit-Queue: Sonny Rao <sonnyrao@chromium.org>
Tested-by: Sonny Rao <sonnyrao@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
a4ac7a8f67d61f8a616724f9426dbc0fce71ce69 06-Aug-2015 Eric Caruso <ejcaruso@chromium.org> autotest: move constants to common_lib

This file gets imported from lots of server-side tests and
infrastructure so it shouldn't be hiding in client/cros. This
moves the file and fixes all imports.

BUG=chromium:485834
TEST=run many tests, make sure there are no problems importing
constants

Change-Id: I5c2b2a0bef6ebc998e62f40162a6ec9cc4436480
Reviewed-on: https://chromium-review.googlesource.com/291302
Trybot-Ready: Eric Caruso <ejcaruso@chromium.org>
Commit-Queue: Eric Caruso <ejcaruso@chromium.org>
Tested-by: Eric Caruso <ejcaruso@chromium.org>
Reviewed-by: Chris Sosa <sosa@google.com>
/external/autotest/client/cros/multimedia/display_facade_native.py
d062d2c16e8fdd934d3eb85eb28d9bf116ce9bca 07-Apr-2015 Tom Wai-Hong Tam <waihong@chromium.org> Chameleon: Fix is_mirrored_enabled and use composition in place of inheritance

A previous patch 262202 makes is_mirrored_enabled() work on client tests
by treating the return value of get_display_info() as an array of DisplayInfo
objects, not an array of dicts.

The get_display_info() returns a native type in DisplayFacadeNative and
casts it to an array of DisplayInfo objects in DisplayFacadeLocalAdapter
(for client tests) and DisplayFacadeRemoteAdapter (for server tests) too.

The patch works in client tests because DisplayFacadeLocalAdapter
inherits DisplayFacadeNative and the get_display_info() is polymorphic.

However, it breaks server tests as get_display_info() is no longer
polymorphic. To avoid this inconsistent behavior, DisplayFacadeLocalAdapter
should avoid inheritance and use composition instead.

BUG=chromium:473950
TEST=Ran display_LidCloseOpen.mirrored and display_ClientChameleonConnection
passed on peach_pit.

Change-Id: I2657596163ebe161deb5e19eea0646441e64e5e3
Reviewed-on: https://chromium-review.googlesource.com/264428
Trybot-Ready: Wai-Hong Tam <waihong@chromium.org>
Tested-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Mussa Kiroga <mussa@chromium.org>
Reviewed-by: Hung-ying Tyan <tyanh@chromium.org>
Commit-Queue: Wai-Hong Tam <waihong@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
32a15dd607c55ed227b47591facd637b86fcb801 24-Mar-2015 Mussa <mussa@chromium.org> Use Chameleon to capture video frames for image comp tests.

Chameleon helps us capture video frames during a video playback.

Generating golden images will be done as a next step before
enabling the test.

BUG=chromium:418217
TEST=Locally tested.

Change-Id: I9995d6d795cc2fade0f0d20813120d955d131bc6
Reviewed-on: https://chromium-review.googlesource.com/262202
Reviewed-by: Wai-Hong Tam <waihong@chromium.org>
Commit-Queue: Mussa Kiroga <mussa@chromium.org>
Tested-by: Mussa Kiroga <mussa@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
2452ae078a86d052642a615f024a6403c34c0cb4 23-Mar-2015 Hung-ying Tyan <tyanh@chromium.org> Chameleon: fix get_internal_resolution()

Currently, display_facade_native.get_internal_resolution() gets the
resolution of the internal screen from the system tool which may not
be the same as the one that's conveyed to the user. For example, on
samus, the raw resolution of the internal screen is 2560x1700 while
the user perceived one is 1280x850.

The problem when this occurs is that we will generate the test image
of the wrong size and then we will make wrong decision about whether
DUT outputs at the TV level. This can be reproduced by running
display_Resolution.mirrored on samus.

In this commit, we get the resolution from the display panel of Chrome
instead of relying on the system tool. It's the reliable way to get
the user perceived resolution of the screen.

BUG=chromium:469553
TEST=successfully ran display_Resolution.mirrored on samus, peach_pi
and peppy with related change 1d7651e.

Change-Id: If227476b493ea6412b3ea70eeb73c336fc7c0665
Reviewed-on: https://chromium-review.googlesource.com/261817
Tested-by: Hung-ying Tyan <tyanh@chromium.org>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>
Commit-Queue: Hung-ying Tyan <tyanh@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
e2693889d1626cae746ac235d20f4b9f9658b326 17-Mar-2015 Hung-ying Tyan <tyanh@chromium.org> Chameleon: fix set_resolution() on link

Looks like when setDisplayMode() in Chrome is being executed (asynchronously),
invoking 'modetest' to check status would cause setDisplayMode() to
malfunction.

This commit checks the status in Chrome instead, and removes the legacy code
for pre-M38 releases.

BUG=chromium:467878
TEST=Ran display_ResolutionList.extended on link (Freon) over both HDMI and DP,
and on blaze (X) over HDMI. Observed that set_resolution() worked in all cases.

Change-Id: I2d4ff6872947cb325976f4b81b2e3f6c7b7d842c
Reviewed-on: https://chromium-review.googlesource.com/260512
Tested-by: Hung-ying Tyan <tyanh@chromium.org>
Reviewed-by: Hung-ying Tyan <tyanh@chromium.org>
Commit-Queue: Hung-ying Tyan <tyanh@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
54cdcdfb4c1d82b8648dee554d74370a1b73f7e1 04-Mar-2015 Hung-ying Tyan <tyanh@chromium.org> Chameleon: add back xrandr implementation

http://crosreview.com/250674 converts chameleon tests to Freon but the tests
fail to run on other platforms that have not enabled Freon. This change brings
the xrandr implementation back in order for those platforms to run chameleon
tests.

BUG=chromium:423527
TEST=Successfully ran suite:chameleon_hdmi on peppy (freon), big (X) and pi (X).
CQ-DEPEND=CL:250674

Change-Id: Ia473690e4d754ba39d7de91cfdd90b6f40089518
Reviewed-on: https://chromium-review.googlesource.com/257984
Reviewed-by: Ilja Friedel <ihf@chromium.org>
Trybot-Ready: Hung-ying Tyan <tyanh@chromium.org>
Tested-by: Hung-ying Tyan <tyanh@chromium.org>
Commit-Queue: Hung-ying Tyan <tyanh@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
d94feb65407b07bd2c17ffd94cfcace57f9d0650 17-Feb-2015 Ilja H. Friedel <ihf@chromium.org> Convert chameleon tests to Freon.

TEST=Ran on X and Freon variant of parrot
test_that --args "chameleon_host=$CHAMELEON_IP" $DUT_IP suite:chameleon_hdmi
BUG=chromium:423527, chromium:451787, chromium:451800
CQ-DEPEND=CL:242131

Change-Id: I9f145c387a22529b369e6017894ae20c5f6f429f
Reviewed-on: https://chromium-review.googlesource.com/250674
Tested-by: Ilja Friedel <ihf@chromium.org>
Reviewed-by: Hung-ying Tyan <tyanh@chromium.org>
Commit-Queue: Ilja Friedel <ihf@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
704134ddfa2e1d67afbfabc7afcc24f849ac0e36 06-Mar-2015 Achuith Bhandarkar <achuith@chromium.org> Use telemetry.core.exceptions.Error everywhere.

BUG=chromium:460625
TEST=bots

Change-Id: I819b78ea402ee8bd8c669ec471c840106f35e548
Reviewed-on: https://chromium-review.googlesource.com/256740
Trybot-Ready: Achuith Bhandarkar <achuith@chromium.org>
Tested-by: Achuith Bhandarkar <achuith@chromium.org>
Reviewed-by: Wai-Hong Tam <waihong@chromium.org>
Commit-Queue: Achuith Bhandarkar <achuith@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
fab82d1625836e41617794f6e2c64dfb019991af 26-Feb-2015 Achuith Bhandarkar <achuith@chromium.org> Remove references to util.TimeoutException

https://codereview.chromium.org/952693003 moved TimeoutException.

chrome.py will need to reference the new location of TimeoutException when the above CL lands.

BUG=chromium:460625
TEST=login_ChromeProfileSanitary, bots

Change-Id: I00a2a7fc9cffacf23b306043296c2b5d6dd895c1
Reviewed-on: https://chromium-review.googlesource.com/254220
Reviewed-by: Toni Barzic <tbarzic@chromium.org>
Commit-Queue: Achuith Bhandarkar <achuith@chromium.org>
Trybot-Ready: Achuith Bhandarkar <achuith@chromium.org>
Tested-by: Achuith Bhandarkar <achuith@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
1338244320c57bbd1c9f1c02ffc083bb7b234ac5 27-Jan-2015 Tom Wai-Hong Tam <waihong@chromium.org> Chameleon: Capture screenshot from a tab in Freon instead of framebuffer

In Freon, the framebuffer capture is still in progress. This change
is to capture screenshot from a tab such that the original tests
are not broken.

BUG=chromium:423527
TEST=Ran display_Resolution.extended on Freon-enabled Link

Change-Id: Ia1e2e063881cc0373c3aa63c95fdcdff25083e84
Reviewed-on: https://chromium-review.googlesource.com/245963
Reviewed-by: Wai-Hong Tam <waihong@chromium.org>
Tested-by: Wai-Hong Tam <waihong@chromium.org>
Commit-Queue: Wai-Hong Tam <waihong@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
39a3ac1e10e8aa28e4b87d85b402bcf3048c340f 03-Feb-2015 Tom Wai-Hong Tam <waihong@chromium.org> Chameleon: Increase the delay of retrying display command

The display query command, i.e. xrandr, is flaky when calling it
frequently to change the HDCP setting. The root cause is unknown.
Increasing the delay between solves this problem.

BUG=chromium:447497
TEST=Ran display_HDCPScreen.extended 100 time and got 93% pass-rate.

Change-Id: Ib85ac84426a98a7868c6173273ef649104f24833
Reviewed-on: https://chromium-review.googlesource.com/245862
Tested-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>
Commit-Queue: Wai-Hong Tam <waihong@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
055ada6014f9c5808c616a2d1417b3261e2e1b10 03-Feb-2015 Tom Wai-Hong Tam <waihong@chromium.org> Chameleon: Retry on more Chrome exceptions

Sometime Chrome is a bit flaky and raises an exception. Retry the
calls again on the exception.

BUG=chromium:451632
TEST=Ran display_HotPlugAtBoot.extended test several time.

Change-Id: Ic65e47ea53d49b3dc575ba9bf64be903c7268fa0
Reviewed-on: https://chromium-review.googlesource.com/245581
Tested-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>
Commit-Queue: Wai-Hong Tam <waihong@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
da4a85a8fefbfe3ee7bbf09629eef2522fa3d178 19-Dec-2014 Tom Wai-Hong Tam <waihong@chromium.org> Chameleon: Implement the display_HDCPScreen test

This new test display_HDCPScreen remotely forces CrOS to enable HDCP and
compares screens between CrOS and Chameleon.

BUG=chromium:418253
TEST=Ran display_HDCPScreen on Pit, Big, and Peppy (TestNAError).

Change-Id: Ib224e9aac2ac41debc2cf5218c111322caf45a2c
Reviewed-on: https://chromium-review.googlesource.com/236760
Tested-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Hung-ying Tyan <tyanh@chromium.org>
Commit-Queue: Wai-Hong Tam <waihong@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
a1fd56118f68ac35394c2bd48d489a9a878714f8 12-Dec-2014 Tom Wai-Hong Tam <waihong@chromium.org> Chameleon: Retry the load_url and close_tab calls which use Chrome

This 2 methods use Chrome to open and close a tab. The Chrome call
path may not be ready immediately on resuming from suspending.

BUG=chromium:435416
TEST=Ran the display_HotPlugAtSuspend.extended on peach_pi.
TEST=Ran the display_ResolutionList.extended on peach_pi.

Change-Id: I92f65531d2cdb72ed3d37d8ec392307fd625ef23
Reviewed-on: https://chromium-review.googlesource.com/235445
Tested-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>
Commit-Queue: Wai-Hong Tam <waihong@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
37f8e83e06bcc3a9bae2137e9d29c7c1086c11fb 10-Dec-2014 Tom Wai-Hong Tam <waihong@chromium.org> Chameleon: Fix accessing to the raw display_info which is a dict

This issue is caused by CL:232136 which moved a method to the
DisplayFacadeNative class. In the DisplayFacadeNative class,
the returned display_info is a dict, still not converted to a
DisplayInfo object. Fix the way to access the properties.

BUG=chromium:440925
TEST=None, simple fix.

Change-Id: I2cd91c9fa592adb763d72d8c7b413f6d2dd5cf06
Reviewed-on: https://chromium-review.googlesource.com/234451
Tested-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>
Commit-Queue: Wai-Hong Tam <waihong@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
7a49d75545994c991356899af9f9ae9a355a62eb 26-Nov-2014 Tom Wai-Hong Tam <waihong@chromium.org> Chameleon: Remove retry logic and use library call for check display connected

Use the library call wait_display_connected which has retry logic. Also
improve the method naming.

BUG=chromium:429862
TEST=Ran affected test passed in falco.

Change-Id: I62c93bae791b281cdc45dfc6911ab350817c1eae
Reviewed-on: https://chromium-review.googlesource.com/232142
Tested-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Hung-ying Tyan <tyanh@chromium.org>
Commit-Queue: Wai-Hong Tam <waihong@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
18b08921a87efd1da6eec8cd1e1cd6edeab15b8e 27-Nov-2014 Tom Wai-Hong Tam <waihong@chromium.org> Chameleon: Move is_display_primary call to display facade library

Moving it to display facade increases reusability, for supporting client
tests in addition.

BUG=chromium:429862
TEST=None

Change-Id: I1b4e8f39f64435f9976cf7ec71a88fc1fc541d6d
Reviewed-on: https://chromium-review.googlesource.com/232136
Reviewed-by: Wai-Hong Tam <waihong@chromium.org>
Commit-Queue: Wai-Hong Tam <waihong@chromium.org>
Tested-by: Wai-Hong Tam <waihong@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
06df6022edd60e7aff06c29ff86998aa2a31afb7 22-Nov-2014 Tom Wai-Hong Tam <waihong@chromium.org> Chameleon: Refactor the call to get the first external display index

This change makes the call elemental to get the first external display
index, instead of the (index, resolutions).

It also moves the logic to the native display facade and removes the
abstraction in the ChameleonTest layer.

BUG=chromium:429862
TEST=Ran the display_ResolutoinList passed in peach_pit.

Change-Id: Ic1ee48a902d92747b64a6c8afa19a7837de6e270
Reviewed-on: https://chromium-review.googlesource.com/231564
Tested-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Hung-ying Tyan <tyanh@chromium.org>
Commit-Queue: Wai-Hong Tam <waihong@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
1c538b18dad43c6883055ac1832002b92749b1c7 25-Nov-2014 Tom Wai-Hong Tam <waihong@chromium.org> Chameleon: Retry the get_output_rect method in which calling xrandr is flaky

We saw a failure that calling xrandr in this method failed. Retrying it
improves its reliability.

BUG=chromium:436295
TEST=Ran display_EdidStress passed on Peppy.

Change-Id: I826004465b984ad4cf1af44e9c5b713ecaa22cf4
Reviewed-on: https://chromium-review.googlesource.com/231732
Tested-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>
Commit-Queue: Wai-Hong Tam <waihong@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
ac108f73f260ff7308d70d7c8be3a5a632847b98 20-Nov-2014 Tom Wai-Hong Tam <waihong@chromium.org> Chameleon: Retry the elemental calls which are flaky

This change moves the retry decorators to the elemental calls, which are
the root of flakiness. So some other calls, like is_mirrored_enabled(),
are retry-enabled.

BUG=chromium:434943
TEST=Ran display_HotPlugNoisy on peach_pit passed.

Change-Id: I77775d0a5484f13468254835274b11f54ca9ee0a
Reviewed-on: https://chromium-review.googlesource.com/231153
Tested-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>
Commit-Queue: Wai-Hong Tam <waihong@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
a30803ab1aba34113b20a5420ad47f25811222e0 11-Nov-2014 Hung-ying Tyan <tyanh@chromium.org> Fix display_facade_native.hide_cursor()

as graphics_utils API changed in f8e316281a8793462e435ce14bceaa22c350a2a6.

BUG=chromium:413085
TEST=run display_Resolution.extended with chameleon and see it passed.

Change-Id: I5cf4dbf28ecc3f05bb3d7b5cb8e84cf5094a08a0
Reviewed-on: https://chromium-review.googlesource.com/228971
Tested-by: Hung-ying Tyan <tyanh@chromium.org>
Reviewed-by: Wai-Hong Tam <waihong@chromium.org>
Commit-Queue: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Ilja Friedel <ihf@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
5780546bf60fa1b8e49528f3f4abac4796f3af6a 08-Nov-2014 Ilja Friedel <ihf@chromium.org> Revert "Revert "autotest: freonize input via uinput.""

As change
https://chromium-review.googlesource.com/228609
has landed we should be able to go back to having the test.

This reverts commit f60483797ceccfe7d7b849c0075028af267bbb50.

BUG=chromium:431429

Change-Id: I7ed770a701f713bb6bda1fc96b66c048fa9f1490
Reviewed-on: https://chromium-review.googlesource.com/228656
Tested-by: Ilja Friedel <ihf@chromium.org>
Reviewed-by: Dan Shi <dshi@chromium.org>
Commit-Queue: Ilja Friedel <ihf@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
f60483797ceccfe7d7b849c0075028af267bbb50 08-Nov-2014 Dan Shi <dshi@chromium.org> Revert "autotest: freonize input via uinput."

This reverts commit f8e316281a8793462e435ce14bceaa22c350a2a6.

BUG=chromium:431429

Change-Id: I4bd92cda6f666f4c4aac1d21cf3c04aa2e49c89a
Reviewed-on: https://chromium-review.googlesource.com/228621
Reviewed-by: Ilja Friedel <ihf@chromium.org>
Commit-Queue: Ilja Friedel <ihf@chromium.org>
Tested-by: Ilja Friedel <ihf@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
f8e316281a8793462e435ce14bceaa22c350a2a6 06-Nov-2014 Ilja H. Friedel <ihf@chromium.org> autotest: freonize input via uinput.

BUG=chromium:413085
TEST=test_that <link_freon> graphics_Stress.gmaps

Change-Id: I0f925188ffa3be1ac6d04a4804d45bfb5bfa1378
Reviewed-on: https://chromium-review.googlesource.com/224885
Tested-by: Ilja Friedel <ihf@chromium.org>
Reviewed-by: Dominik Behr <dbehr@chromium.org>
Commit-Queue: Ilja Friedel <ihf@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py
63303a0c7963646f8c463180d4d25e4de751287f 28-Oct-2014 Tom Wai-Hong Tam <waihong@chromium.org> Chameleon: Improve the naming of classes to access DUT display/audio

This change does the following name changes:

Client side:
AudioUtility -> AudioFacadeNative, only accept native types, for RPC
DisplayUtility -> DisplayFacadeNative, only accept native types, for RPC

Server side:
AudioClient -> AudioFacadeRemoteAdapter, an adapter to AudioFacadeNative
DisplayClient -> DisplayFacadeRemoteAdapter, an adapter to DisplayFacadeNative
MultimediaClientFactory -> RemoteFacadeFactory, to create facade in remote

Also fix some pylint errors.

BUG=chromium:405143
TEST=Ran the chameleon_hdmi suite on Falco.

Change-Id: I9c2517f68236bd033ffb7df3c21629b42956f790
Reviewed-on: https://chromium-review.googlesource.com/226140
Commit-Queue: Wai-Hong Tam <waihong@chromium.org>
Tested-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: Wai-Hong Tam <waihong@chromium.org>
/external/autotest/client/cros/multimedia/display_facade_native.py