Adding option to move suggestion popup item text to omnibox.

chrome/android/shell/java/src/org/chromium/chrome/shell/omnibox/SuggestionArrayAdapter.java

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.chrome.shell.omnibox;
 
 import android.content.Context;
 import android.view.LayoutInflater;
 import android.view.View;
 import android.view.ViewGroup;
 import android.widget.AbsListView.LayoutParams;
 import android.widget.ArrayAdapter;
+import android.widget.EditText;
+import android.widget.ImageView;
 import android.widget.TextView;
 
 import org.chromium.chrome.browser.omnibox.OmniboxSuggestion;
 import org.chromium.chrome.shell.R;
 
 import java.util.List;
 
 /**
  * Adapter that provides suggestion views for the suggestion popup.
  */
 class  SuggestionArrayAdapter extends ArrayAdapter<OmniboxSuggestion> {
     private final List<OmniboxSuggestion> mSuggestions;
+    private EditText mUrlTextView;
 
-    public SuggestionArrayAdapter(Context context, int res, List<OmniboxSuggestion> suggestions) {
+    public SuggestionArrayAdapter(Context context, int res, List<OmniboxSuggestion> suggestions,
+                                  EditText urlTextView) {
         super(context, res, suggestions);
         mSuggestions = suggestions;
+        mUrlTextView = urlTextView;
     }
 
     @Override
     public View getView(int position, View convertView, ViewGroup parent) {
         View v = convertView;
         if (v == null) {
             LayoutInflater vi = (LayoutInflater) getContext().getSystemService(
                     Context.LAYOUT_INFLATER_SERVICE);
-            v = vi.inflate(R.layout.dropdown_item, null);
+            v = vi.inflate(R.layout.suggestion_item, null);
             int height = getContext().getResources().getDimensionPixelSize(
                     R.dimen.dropdown_item_height);
             v.setLayoutParams(new LayoutParams(LayoutParams.MATCH_PARENT, height));
         }
-        TextView t1 = (TextView) v.findViewById(R.id.dropdown_label);
-        t1.setText(mSuggestions.get(position).getDisplayText());
+        TextView t1 = (TextView) v.findViewById(R.id.suggestion_item_label);
+        final String suggestionText = mSuggestions.get(position).getDisplayText();
+        t1.setText(suggestionText);

[Bernhard Bauer, 2014/09/10 07:53:31]

I think you could directly inline this.

[ankit, 2014/09/10 09:15:52]

In onClickListener I am using the same variable suggestionText at 2 places, 
If I inline this then |mSuggestions.get(position).getDisplayText()|
be replicated at all places where suggestionText is being used.
Please suggest if inlining this is fine or it should be as is.

[Bernhard Bauer, 2014/09/10 10:40:34]

Oh, didn't see that. Okay, let's keep it as it is.

 
-        TextView t2 = (TextView) v.findViewById(R.id.dropdown_sublabel);
+        TextView t2 = (TextView) v.findViewById(R.id.suggestion_item_sublabel);
         t2.setText(mSuggestions.get(position).getUrl());
+
+        ImageView arrow = (ImageView) v.findViewById(R.id.suggestion_item_arrow);
+        arrow.setOnClickListener(new View.OnClickListener() {
+            @Override
+            public void onClick(View v) {
+                mUrlTextView.setText(suggestionText);
+                mUrlTextView.setSelection(suggestionText.length());
+            }
+        });
         return v;
     }
 }