Bug 22757 - break out text caret paint as enabled feature
Summary: break out text caret paint as enabled feature
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other Linux
: P2 Normal
Assignee: Adam Barth
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-12-09 08:35 PST by Cary Clark
Modified: 2008-12-17 13:40 PST (History)
1 user (show)

See Also:


Attachments
patch to allow disabling text caret (4.11 KB, patch)
2008-12-09 08:38 PST, Cary Clark
darin: review+
Details | Formatted Diff | Diff
patch to allow disabling text caret (4.15 KB, patch)
2008-12-09 09:19 PST, Cary Clark
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Cary Clark 2008-12-09 08:35:15 PST
Android draws input text in a separate thread using a java-based text view. Suppress drawing the caret when the feature is disabled and a text field is selected, to reduce the painting and invalidates that Android responds to.
Comment 1 Cary Clark 2008-12-09 08:38:22 PST
Created attachment 25887 [details]
patch to allow disabling text caret

one of many patches to get local Android changes into webkit
Comment 2 Darin Adler 2008-12-09 09:06:00 PST
Comment on attachment 25887 [details]
patch to allow disabling text caret

> Index: JavaScriptCore/ChangeLog
> ===================================================================
> --- JavaScriptCore/ChangeLog	(revision 39135)
> +++ JavaScriptCore/ChangeLog	(working copy)
> @@ -1,3 +1,13 @@
> +2008-12-09  Cary Clark  <caryclark@google.com>
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +	Add ENABLE_TEXT_CARET to permit the ANDROID platform
> +	to invalidate and draw the caret in a separate thread.
> +
> +        * wtf/Platform.h:
> +	Default ENABLE_TEXT_CARET to 1.

ChangeLog has tabs in it. This makes it harder to land the patch, because the committer has to change the tabs to spaces. Please don't use tabs in WebKit changes.

Patch otherwise looks fine. r=me
Comment 3 Cary Clark 2008-12-09 09:19:48 PST
Created attachment 25889 [details]
patch to allow disabling text caret

replace tabs with spaces in ChangeLogs
Comment 4 Adam Barth 2008-12-17 11:22:10 PST
Will land.
Comment 5 Adam Barth 2008-12-17 13:40:30 PST
Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	JavaScriptCore/ChangeLog
	M	JavaScriptCore/wtf/Platform.h
	M	WebCore/ChangeLog
	M	WebCore/page/Frame.cpp
Committed r39362
	M	JavaScriptCore/wtf/Platform.h
	M	JavaScriptCore/ChangeLog
	M	WebCore/ChangeLog
	M	WebCore/page/Frame.cpp
r39362 = ed04161a85f1c3cffaec802851b6a437af34d68d (trunk)