Chromium Code Reviews| Index: chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellActivity.java |
| diff --git a/chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellActivity.java b/chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellActivity.java |
| index ab0f53324a519fb38e0ba3038ec04df633bd1ebb..b0d4385177f17f39affb61a29f434d83375cdd07 100644 |
| --- a/chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellActivity.java |
| +++ b/chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellActivity.java |
| @@ -13,6 +13,7 @@ import android.view.KeyEvent; |
| import android.view.Menu; |
| import android.view.MenuItem; |
| import android.view.View; |
| +import android.view.ViewGroup; |
| import android.widget.Toast; |
| import org.chromium.base.ApiCompatibilityUtils; |
| @@ -291,6 +292,11 @@ public class ChromeShellActivity extends Activity implements AppMenuPropertiesDe |
| @Override |
| public boolean onOptionsItemSelected(MenuItem item) { |
| ChromeShellTab activeTab = getActiveTab(); |
| + ViewGroup containerView = activeTab.getContentViewCore().getContainerView(); |
| + if (activeTab != null && containerView.isFocusable() |
|
Bernhard Bauer
2014/09/15 13:52:56
You are accessing |activeTab| on the previous line
ankit
2014/09/15 14:19:50
Sorry for such mistake.
|
| + && containerView.isFocusableInTouchMode()) { |
| + containerView.requestFocus(); |
| + } |
| switch (item.getItemId()) { |
| case R.id.signin: |
| if (ChromeSigninController.get(this).isSignedIn()) { |