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

Unified Diff: platform_tools/android/apps/viewer/src/main/java/org/skia/viewer/StateAdapter.java

Issue 2043793002: Show FPS in UI state (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: line100 Created 4 years, 6 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 | « no previous file | platform_tools/android/apps/viewer/src/main/res/values/integers.xml » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: platform_tools/android/apps/viewer/src/main/java/org/skia/viewer/StateAdapter.java
diff --git a/platform_tools/android/apps/viewer/src/main/java/org/skia/viewer/StateAdapter.java b/platform_tools/android/apps/viewer/src/main/java/org/skia/viewer/StateAdapter.java
index 4c9dcd4b847dbca88eb0706ee3fa3d9152ae5807..d546c7bb3e19d520baff1a61efeaebb0ac85621c 100644
--- a/platform_tools/android/apps/viewer/src/main/java/org/skia/viewer/StateAdapter.java
+++ b/platform_tools/android/apps/viewer/src/main/java/org/skia/viewer/StateAdapter.java
@@ -6,6 +6,7 @@ import android.view.ViewGroup;
import android.widget.AdapterView;
import android.widget.ArrayAdapter;
import android.widget.BaseAdapter;
+import android.widget.LinearLayout;
import android.widget.Spinner;
import android.widget.TextView;
@@ -15,12 +16,23 @@ import org.json.JSONObject;
import java.util.ArrayList;
+/*
+ The navigation drawer requires ListView, so we implemented this BaseAdapter for that ListView.
+ However, the ListView does not provide good support for updating just a single child view.
+ For example, a frequently changed child view such as FPS state will reset the spinner of
+ all other child views; although I didn't change other child views and directly return
+ the convertView in BaseAdapter.getView(int position, View convertView, ViewGroup parent).
+
+ Therefore, our adapter only returns one LinearLayout for the ListView.
+ Within that LinearLayout, we maintain views ourselves so we can efficiently update its children.
+ */
public class StateAdapter extends BaseAdapter implements AdapterView.OnItemSelectedListener {
static final String NAME = "name";
static final String VALUE = "value";
static final String OPTIONS = "options";
ViewerActivity mViewerActivity;
+ LinearLayout mLayout;
JSONArray mStateJson;
public StateAdapter(ViewerActivity viewerActivity) {
@@ -36,7 +48,11 @@ public class StateAdapter extends BaseAdapter implements AdapterView.OnItemSelec
public void setState(String stateJson) {
try {
mStateJson = new JSONArray(stateJson);
- notifyDataSetChanged();
+ if (mLayout != null) {
+ updateDrawer();
+ } else {
+ notifyDataSetChanged();
+ }
} catch (JSONException e) {
e.printStackTrace();
}
@@ -44,17 +60,12 @@ public class StateAdapter extends BaseAdapter implements AdapterView.OnItemSelec
@Override
public int getCount() {
- return mStateJson.length();
+ return 1;
}
@Override
public Object getItem(int position) {
- try {
- return mStateJson.getJSONObject(position);
- } catch (JSONException e) {
- e.printStackTrace();
- return null;
- }
+ return null;
}
@Override
@@ -64,39 +75,66 @@ public class StateAdapter extends BaseAdapter implements AdapterView.OnItemSelec
@Override
public View getView(int position, View convertView, ViewGroup parent) {
- if (convertView == null) {
- convertView = LayoutInflater.from(mViewerActivity).inflate(R.layout.state_item, null);
+ if (mLayout == null) {
+ mLayout = new LinearLayout(mViewerActivity);
+ mLayout.setOrientation(LinearLayout.VERTICAL);
+ updateDrawer();
}
- TextView nameText = (TextView) convertView.findViewById(R.id.nameText);
- TextView valueText = (TextView) convertView.findViewById(R.id.valueText);
- Spinner optionSpinner = (Spinner) convertView.findViewById(R.id.optionSpinner);
- JSONObject stateObject = (JSONObject) getItem(position);
- try {
- nameText.setText(stateObject.getString(NAME));
- String value = stateObject.getString(VALUE);
- JSONArray options = stateObject.getJSONArray(OPTIONS);
- if (options.length() == 0) {
- valueText.setText(value);
- valueText.setVisibility(View.VISIBLE);
- optionSpinner.setVisibility(View.GONE);
+ return mLayout;
+ }
- } else {
- ArrayList<String> optionList = new ArrayList<>();
- String[] optionStrings = new String[options.length()];
- for(int i=0; i<options.length(); i++) {
- optionList.add(options.getString(i));
+ private View inflateItemView(JSONObject item) throws JSONException {
+ View itemView = LayoutInflater.from(mViewerActivity).inflate(R.layout.state_item, null);
+ TextView nameText = (TextView) itemView.findViewById(R.id.nameText);
+ TextView valueText = (TextView) itemView.findViewById(R.id.valueText);
+ Spinner optionSpinner = (Spinner) itemView.findViewById(R.id.optionSpinner);
+ nameText.setText(item.getString(NAME));
+ String value = item.getString(VALUE);
+ JSONArray options = item.getJSONArray(OPTIONS);
+ if (options.length() == 0) {
+ valueText.setText(value);
+ valueText.setVisibility(View.VISIBLE);
+ optionSpinner.setVisibility(View.GONE);
+
+ } else {
+ ArrayList<String> optionList = new ArrayList<>();
+ String[] optionStrings = new String[options.length()];
+ for (int j = 0; j < options.length(); j++) {
+ optionList.add(options.getString(j));
+ }
+ optionSpinner.setAdapter(new ArrayAdapter<String>(mViewerActivity,
+ android.R.layout.simple_spinner_dropdown_item, optionList));
+ optionSpinner.setSelection(optionList.indexOf(value));
+ optionSpinner.setOnItemSelectedListener(this);
+ optionSpinner.setVisibility(View.VISIBLE);
+ valueText.setVisibility(View.GONE);
+ }
+ itemView.setTag(item.toString()); // To save unnecessary view update
+ itemView.setTag(R.integer.value_tag_key, value); // To save unnecessary state change event
+ return itemView;
+ }
+
+ private void updateDrawer() {
+ try {
+ if (mStateJson.length() < mLayout.getChildCount()) {
+ mLayout.removeViews(
+ mStateJson.length(), mLayout.getChildCount() - mStateJson.length());
+ }
+ for (int i = 0; i < mStateJson.length(); i++) {
+ JSONObject stateObject = mStateJson.getJSONObject(i);
+ if (mLayout.getChildCount() > i) {
+ View childView = mLayout.getChildAt(i);
+ if (stateObject.toString().equals(childView.getTag())) {
+ continue; // No update, reuse the old view and skip the remaining step
+ } else {
+ mLayout.removeViewAt(i);
+ }
}
- optionSpinner.setAdapter(new ArrayAdapter<String>(mViewerActivity,
- android.R.layout.simple_spinner_dropdown_item, optionList));
- optionSpinner.setSelection(optionList.indexOf(value));
- optionSpinner.setOnItemSelectedListener(this);
- optionSpinner.setVisibility(View.VISIBLE);
- valueText.setVisibility(View.GONE);
+ mLayout.addView(inflateItemView(stateObject), i);
}
} catch (JSONException e) {
e.printStackTrace();
}
- return convertView;
}
@Override
@@ -104,7 +142,10 @@ public class StateAdapter extends BaseAdapter implements AdapterView.OnItemSelec
View stateItem = (View) parent.getParent();
String stateName = ((TextView) stateItem.findViewById(R.id.nameText)).getText().toString();
String stateValue = ((TextView) view).getText().toString();
- mViewerActivity.onStateChanged(stateName, stateValue);
+ if (!stateValue.equals(stateItem.getTag(R.integer.value_tag_key))) {
+ stateItem.setTag(null); // Reset the tag to let updateDrawer update this item view.
+ mViewerActivity.onStateChanged(stateName, stateValue);
+ }
}
@Override
« no previous file with comments | « no previous file | platform_tools/android/apps/viewer/src/main/res/values/integers.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698