diff options
| author | Michael Kolb <kolby@google.com> | 2011-08-15 15:37:30 -0700 |
|---|---|---|
| committer | Michael Kolb <kolby@google.com> | 2011-08-15 16:17:26 -0700 |
| commit | 8a009498bfca64556c2082cbb32f68506d6962db (patch) | |
| tree | 01bbdc884cbfcffba6addd77ca362dcc8772dcf4 /src/com/android/browser/Controller.java | |
| parent | c3af06776be83ba64a0d3549cb72ca6e5e7f03cd (diff) | |
Fix monkey bug
Bug: 5166733
added missing return statements and valid active
tab check
Change-Id: Ida7ffa6faf77a95eaf19ef566176ad522be4efef
Diffstat (limited to 'src/com/android/browser/Controller.java')
| -rw-r--r-- | src/com/android/browser/Controller.java | 19 |
1 files changed, 14 insertions, 5 deletions
diff --git a/src/com/android/browser/Controller.java b/src/com/android/browser/Controller.java index 3f126c07..24072c9e 100644 --- a/src/com/android/browser/Controller.java +++ b/src/com/android/browser/Controller.java @@ -2684,23 +2684,32 @@ public class Controller } public boolean dispatchKeyEvent(KeyEvent event) { - return mBlockEvents; + return mBlockEvents || hasNoActiveTab(); } public boolean dispatchKeyShortcutEvent(KeyEvent event) { - return mBlockEvents; + return mBlockEvents || hasNoActiveTab(); } public boolean dispatchTouchEvent(MotionEvent ev) { - return mBlockEvents; + return mBlockEvents || hasNoActiveTab(); } public boolean dispatchTrackballEvent(MotionEvent ev) { - return mBlockEvents; + return mBlockEvents || hasNoActiveTab(); } public boolean dispatchGenericMotionEvent(MotionEvent ev) { - return mBlockEvents; + return mBlockEvents || hasNoActiveTab(); + } + + private boolean hasNoActiveTab() { + Tab tab = getCurrentTab(); + if (tab == null) { + Log.w(LOGTAG, "Received event with no active tab. Tab count: " + + mTabControl.getTabCount()); + } + return tab == null; } } |
