diff options
author | Leon Scroggins <scroggo@google.com> | 2011-03-09 16:16:50 -0500 |
---|---|---|
committer | Leon Scroggins <scroggo@google.com> | 2011-03-09 16:16:50 -0500 |
commit | 65e0982620f2ebb654d490694601d15f6d90b546 (patch) | |
tree | 6756cb640a0f18c763e085c30ed01699d6f85117 /WebKit/android/jni | |
parent | 615fcf00641400b4851396f3ac5525818c4aa33a (diff) | |
download | external_webkit-65e0982620f2ebb654d490694601d15f6d90b546.zip external_webkit-65e0982620f2ebb654d490694601d15f6d90b546.tar.gz external_webkit-65e0982620f2ebb654d490694601d15f6d90b546.tar.bz2 |
Do not attempt to put textfield onscreen when size changes.
Bug:4079218
A frameworks/base change will make it so the textfield scrolls
on screen when the IME is shown.
Change-Id: I654079288fdad657f476948476d689359cf48804
Diffstat (limited to 'WebKit/android/jni')
-rw-r--r-- | WebKit/android/jni/WebViewCore.cpp | 11 | ||||
-rw-r--r-- | WebKit/android/jni/WebViewCore.h | 3 |
2 files changed, 1 insertions, 13 deletions
diff --git a/WebKit/android/jni/WebViewCore.cpp b/WebKit/android/jni/WebViewCore.cpp index 5744609..34179f2 100644 --- a/WebKit/android/jni/WebViewCore.cpp +++ b/WebKit/android/jni/WebViewCore.cpp @@ -344,7 +344,6 @@ WebViewCore::WebViewCore(JNIEnv* env, jobject javaWebViewCore, WebCore::Frame* m #endif m_isPaused = false; m_screenOnCounter = 0; - m_onlyScrollIfImeIsShowing = false; m_shouldPaintCaret = true; LOG_ASSERT(m_mainFrame, "Uh oh, somehow a frameview was made without an initial frame!"); @@ -948,7 +947,7 @@ void WebViewCore::scrollTo(int x, int y, bool animate) JNIEnv* env = JSC::Bindings::getJNIEnv(); env->CallVoidMethod(m_javaGlue->object(env).get(), m_javaGlue->m_scrollTo, - x, y, animate, m_onlyScrollIfImeIsShowing); + x, y, animate, false); checkException(env); } @@ -1316,14 +1315,6 @@ void WebViewCore::setSizeScreenWidthAndScale(int width, int height, } } - // If this was in response to touching a textfield and showing the IME, - // the IME may now cover textfield. Bring it back into view. - // If the scale changed, however, this was the result of a zoom. - if (oldScale == m_scale && osh > screenHeight) { - m_onlyScrollIfImeIsShowing = true; - revealSelection(); - m_onlyScrollIfImeIsShowing = false; - } // update the currently visible screen as perceived by the plugin sendPluginVisibleScreen(); } diff --git a/WebKit/android/jni/WebViewCore.h b/WebKit/android/jni/WebViewCore.h index be8de94..df7169d 100644 --- a/WebKit/android/jni/WebViewCore.h +++ b/WebKit/android/jni/WebViewCore.h @@ -608,9 +608,6 @@ namespace android { int m_blurringNodePointer; int m_lastFocusedSelStart; int m_lastFocusedSelEnd; - // Pass along with a scroll message to tell the UI thread to only - // scroll the page if the IME is showing. - bool m_onlyScrollIfImeIsShowing; PictureSet m_content; // the set of pictures to draw SkRegion m_addInval; // the accumulated inval region (not yet drawn) SkRegion m_rebuildInval; // the accumulated region for rebuilt pictures |