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

Unified Diff: frontend/client/src/autotest/afe/AfeUtils.java

Issue 3554003: Merge remote branch 'cros/upstream' into tempbranch3 (Closed) Base URL: http://git.chromium.org/git/autotest.git
Patch Set: Created 10 years, 3 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
Index: frontend/client/src/autotest/afe/AfeUtils.java
diff --git a/frontend/client/src/autotest/afe/AfeUtils.java b/frontend/client/src/autotest/afe/AfeUtils.java
index 3445b3cc835f5ed41d4071612e5f6ba8b9fc09a9..f435a0a1fe5aba527f4878f1c0c996d1a001dc92 100644
--- a/frontend/client/src/autotest/afe/AfeUtils.java
+++ b/frontend/client/src/autotest/afe/AfeUtils.java
@@ -1,6 +1,6 @@
package autotest.afe;
-import autotest.afe.CreateJobView.JobCreateListener;
+import autotest.afe.create.CreateJobViewPresenter.JobCreateListener;
import autotest.common.JSONArrayList;
import autotest.common.JsonRpcCallback;
import autotest.common.JsonRpcProxy;
@@ -10,6 +10,7 @@ import autotest.common.Utils;
import autotest.common.table.JSONObjectSet;
import autotest.common.ui.NotifyManager;
import autotest.common.ui.RadioChooser;
+import autotest.common.ui.SimplifiedList;
import com.google.gwt.json.client.JSONArray;
import com.google.gwt.json.client.JSONBoolean;
@@ -60,7 +61,7 @@ public class AfeUtils {
protected static String[] getFilteredLabelStrings(boolean onlyPlatforms,
boolean onlyNonPlatforms) {
- assert( !(onlyPlatforms && onlyNonPlatforms));
+ assert !(onlyPlatforms && onlyNonPlatforms);
JSONArray labels = staticData.getData("labels").isArray();
List<String> result = new ArrayList<String>();
for (int i = 0; i < labels.size(); i++) {
@@ -71,8 +72,8 @@ public class AfeUtils {
if (atomicGroup != null) {
name += ATOMIC_GROUP_SUFFIX;
}
- if ((onlyPlatforms && labelIsPlatform) ||
- (onlyNonPlatforms && !labelIsPlatform)) {
+ if (onlyPlatforms && labelIsPlatform ||
+ onlyNonPlatforms && !labelIsPlatform) {
result.add(name);
} else if (!onlyPlatforms && !onlyNonPlatforms) {
if (labelIsPlatform) {
@@ -319,13 +320,21 @@ public class AfeUtils {
}
}
- public static void popualateListBox(ListBox box, String staticDataKey) {
+ public static void populateListBox(ListBox box, String staticDataKey) {
JSONArray options = staticData.getData(staticDataKey).isArray();
for (JSONString jsonOption : new JSONArrayList<JSONString>(options)) {
box.addItem(Utils.jsonToString(jsonOption));
}
}
+ public static void populateListBox(SimplifiedList box, String staticDataKey) {
+ JSONArray options = staticData.getData(staticDataKey).isArray();
+ for (JSONString jsonOption : new JSONArrayList<JSONString>(options)) {
+ String option = Utils.jsonToString(jsonOption);
+ box.addItem(option, option);
+ }
+ }
+
public static void setSelectedItem(ListBox box, String item) {
box.setSelectedIndex(0);
for (int i = 0; i < box.getItemCount(); i++) {
« no previous file with comments | « frontend/client/src/autotest/afe/AfeClient.java ('k') | frontend/client/src/autotest/afe/CheckBoxPanel.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698