summaryrefslogtreecommitdiffstats
path: root/WebCore/page
diff options
context:
space:
mode:
authorLeon Scroggins <>2009-04-14 10:57:12 -0700
committerThe Android Open Source Project <initial-contribution@android.com>2009-04-14 10:57:12 -0700
commit3c8902f607e797e45524ea76cc30ec24c1ed6d2e (patch)
treeb80791b9b0ea64330717b85d19147744ef5b03d4 /WebCore/page
parent76fc394d65b2db7f55e09874fa383a153e2bd128 (diff)
downloadexternal_webkit-3c8902f607e797e45524ea76cc30ec24c1ed6d2e.zip
external_webkit-3c8902f607e797e45524ea76cc30ec24c1ed6d2e.tar.gz
external_webkit-3c8902f607e797e45524ea76cc30ec24c1ed6d2e.tar.bz2
AI 146153: Make the TextDialog invisible, so that we only show WebKit's rendering of the textfield, yet we retain the benefits of having the TextDialog.
Frame.cpp: Removed a line that should not be there anyway. Somehow when merging in Apple's ENABLE(TEXT_CARET) changes we left in a line that disables the blinking Cursor. Platform.h: Let WebKit draw the cursor. RenderThemeAndroid.cpp: Make the active selection color (the color of a selection in an active <textfield> or <textarea>) visible, and match the selection color shown in other places in Android. WebViewCore.cpp: Make the focused node's document's focus controller active. This allows webkit to draw the cursor. TextDialog.java: Set WillNotDraw to true, and remove our background, so that the TextDialog is completely invisible. Note that I have not changed its visibility to View.VISIBLE, which would have other undesired effects (would not accept touch events, for example). Since we are not drawing, remove onPreDraw, which is no longer necessary. In onSelectionChanged, pass a message to webkit telling it that the selection has changed. It is important that the two threads always agree (albeit webkit may be behind) because the user will now only see the selection drawn by webkit. Remove some old flawed code whose purpose was to keep the selection up to date and do not send key events for arrow keys (which would mess up the selection that was correct thanks to onSelectionChanged). BUG=1781070,1712752,916193,1411370,1160999,1775425,1354814 Automated import of CL 146153
Diffstat (limited to 'WebCore/page')
-rw-r--r--WebCore/page/Frame.cpp1
1 files changed, 0 insertions, 1 deletions
diff --git a/WebCore/page/Frame.cpp b/WebCore/page/Frame.cpp
index d379e13..f0cd1ec 100644
--- a/WebCore/page/Frame.cpp
+++ b/WebCore/page/Frame.cpp
@@ -614,7 +614,6 @@ void Frame::selectionLayoutChanged()
bool shouldBlink = m_caretVisible
&& selection()->isCaret() && selection()->isContentEditable();
- shouldBlink = false;
// If the caret moved, stop the blink timer so we can restart with a
// black caret in the new location.
if (caretRectChanged || !shouldBlink)