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

Unified Diff: client/bin/factory_ui

Issue 3151032: Add unique_name test attr, fix UI active test selection, clean up test_list. (Closed) Base URL: http://src.chromium.org/git/autotest.git
Patch Set: sequences get unique_name too Created 10 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « client/bin/factory.py ('k') | client/site_tests/suite_Factory/test_list » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: client/bin/factory_ui
diff --git a/client/bin/factory_ui b/client/bin/factory_ui
index 0c1c89bf0357cc94a84daf0758f60c09b5da964c..ba0767eb2af038c5597a70223f011437a8e48bb6 100755
--- a/client/bin/factory_ui
+++ b/client/bin/factory_ui
@@ -201,8 +201,10 @@ class UiState():
automated sequences, since there is no other window, the no
active test message is replaced with an updated list of
subtest status.'''
- if test == self.active_test:
+ if test is None or test == self.active_test:
return
+ factory.log('UI active test -> %s' %
+ self._status_map.test_db.get_unique_details(test))
self.active_test = test
self._test_widget_box.remove(self._active_test_widget)
active_widget = (test in self._status_map.test_db.seq_test_set
« no previous file with comments | « client/bin/factory.py ('k') | client/site_tests/suite_Factory/test_list » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698