Bug 117347 – [rendering] Use foreground color to render the overtype caret

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Claudio Saavedra]

[rendering] Use foreground color to render the overtype caret

[Claudio Saavedra]

Created attachment 204042 [details]
Patch

[Darin Adler]

Comment on attachment 204042 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=204042&action=review

> Source/WebCore/rendering/RenderObject.cpp:1640
> +                color = frame()->selection()->isFocusedAndActive() ?
>                      theme()->activeSelectionBackgroundColor() :
>                      theme()->inactiveSelectionBackgroundColor();

WebKit style guide says to put the "?" and ":" at the beginnings of lines, not the ends of lines, or this fits fine on one line, I think.

> Source/WebCore/rendering/RenderView.h:344
> +    bool m_selectionWasCaret;

It would be better to initialize this when in the RenderView constructor, even if the code will work OK with an uninitialized boolean. If nothing else, the uninitialized read it will irritate people using memory tools like valgrind.

[Claudio Saavedra]

Committed r151322: <http://trac.webkit.org/changeset/151322>