| 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_adapter.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_adapter.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_adapter.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_adapter.py
|
| 1bd677ffd679310202e2868c71b255f744d5d2f9 |
|
10-Sep-2015 |
Chun-Huang Lin <chinlin@google.com> |
Chameleon: Fix screenshot functions for client tests.
http://crosreview.com/257984 added back xrandr implementation
and modified screenshot functions for server tests, but it
forgot to modified those for client tests. This patch fixes
the mistake.
BUG=chromium:529907
TEST=None
Change-Id: Ic8db1a4530470663f90e9178aae38734148bd5fd
Reviewed-on: https://chromium-review.googlesource.com/298753
Reviewed-by: Hung-ying Tyan <tyanh@chromium.org>
Tested-by: Chin-Huang Lin <chinlin@google.com>
/external/autotest/client/cros/multimedia/display_facade_adapter.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_adapter.py
|
| 2c4a5d849e715f8046c4da7d4d4a0e1c50393f14 |
|
11-Aug-2015 |
Chin-Huang Lin <chinlin@google.com> |
Chameleon: Force to convert the image to RGB color format
When capturing the screen image from X using import, it may use a
colormap format in order to reduce file size, if the total number of
colors is not too much. However, it makes the image comparison failed
as it is a different format, not RGB. So force to convert it to RGB.
BUG=chromium:397284
TEST=Ran display_Resolution.mirrord on nyan_big passed.
Change-Id: I4ab9494615a52914f49b414f530112a33c4b8f1d
Reviewed-on: https://chromium-review.googlesource.com/292327
Reviewed-by: Yuli Huang <yuli@chromium.org>
Tested-by: Chin-Huang Lin <chinlin@google.com>
Commit-Queue: Chin-Huang Lin <chinlin@google.com>
Trybot-Ready: Chin-Huang Lin <chinlin@google.com>
/external/autotest/client/cros/multimedia/display_facade_adapter.py
|
| aeee12fc6bc5af104076fd3c93a995c5474a9169 |
|
05-May-2015 |
Hung-ying Tyan <tyanh@chromium.org> |
Chameleon: raise error when set_mirrored() fails
We should fail the test when set_mirrored() fails. Otherwise, it would
take more time to diagnose the problem when the error propagates to a
later stage. In this commit, an error.TestError is raised when the call
fails.
BUG=chromium:482363
TEST=ran display_SwitchMode and observed the error being raised.
Change-Id: Id3a32f114f8b1854359edf7f5b5ae5904be8e07b
Reviewed-on: https://chromium-review.googlesource.com/269302
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_adapter.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_adapter.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_adapter.py
|
| afad1cd381b748dc0dd3ceb4662d413936743b41 |
|
02-Dec-2014 |
Tom Wai-Hong Tam <waihong@chromium.org> |
Fix the display facade local adapter methods which point to its super class
There is no proxy object for the local adapter. Methods should point to
its super class'es methods.
BUG=chromium:405143
TEST=Manual test the methods in the local adapter.
Change-Id: I25846938f065a4fcbdf29264c33714ee74c24858
Reviewed-on: https://chromium-review.googlesource.com/232730
Reviewed-by: Hung-ying Tyan <tyanh@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_adapter.py
|
| 619f336b5e4a78f70d84999116fa458cd03fc05d |
|
27-Nov-2014 |
Tom Wai-Hong Tam <waihong@chromium.org> |
Chameleon: Directly call the display_facade for suspend_resume
Remove the suspend_resume method on the intermedia layer. Change tests
to use the display_facade for suspend_resume.
BUG=chromium:429862
TEST=Ran display_Resolution and display_SuspendStress passed.
Change-Id: Ic9db4ee6bc778f74de3e2cef4e4b2dfd5d6793f1
Reviewed-on: https://chromium-review.googlesource.com/232146
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_adapter.py
|
| e535050c22ff1651a505e92286886b125f0071e0 |
|
26-Nov-2014 |
Tom Wai-Hong Tam <waihong@chromium.org> |
Rename display_help module to display_info
Use a more particular name.
BUG=chromium:429862
TEST=Ran a Chameleon test passed.
Change-Id: I671efe9cd53780d035933d0f5e63c90639aabaef
Reviewed-on: https://chromium-review.googlesource.com/232144
Reviewed-by: Kalin Stoyanov <kalin@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_adapter.py
|
| 6fa56a06183a3fe96be469b72e700db9a31eab74 |
|
28-Oct-2014 |
Tom Wai-Hong Tam <waihong@chromium.org> |
Chameleon: Implement the local facade factory
The LocalFacadeFactory is able to generate the facade on DUT. It is used
by client tests to access the local audio/display functionality.
BUG=chromium:405143
TEST=None, pending tests to the following CL.
Change-Id: I54a8b972ae77032d502996a4988ccc08a28eca37
Reviewed-on: https://chromium-review.googlesource.com/226141
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_adapter.py
|