[Webkit-unassigned] [Bug 112426] [WK2] Add support for selectTrailingWhitespaceEnabled setting

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=112426





--- Comment #7 from Manuel Rego Casasnovas <rego at igalia.com>  2013-05-16 00:50:47 PST ---
(In reply to comment #6)
> (From update of attachment 195954 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=195954&action=review
> 
> I am confused. How does added two preferences, but not use them anywhere, fixes the tests?

The tests were already fixed in r144221 and r145849, where you can see changes like:

-    testRunner.setSmartInsertDeleteEnabled(false); 
-    internals.settings.setSmartInsertDeleteEnabled(false); 

or:

-     testRunner.setSelectTrailingWhitespaceEnabled(true);
+     internals.settings.setSelectTrailingWhitespaceEnabled(true);

This patch just adds support for selectTrailingWhitespaceEnabled in WK2 as smartInsertDeleteEnabled is already supported. In WK1 both are already supported.

> > Source/WebKit2/WebProcess/WebPage/WebPage.cpp:3842
> > +bool WebPage::isSelectTrailingWhitespaceEnabled()
> > +{
> > +    return m_page->settings()->selectTrailingWhitespaceEnabled();
> > +}
> > +
> > +void WebPage::setSelectTrailingWhitespaceEnabled(bool enabled)
> > +{
> > +    m_page->settings()->setSelectTrailingWhitespaceEnabled(enabled);
> > +}
> > +
> 
> This code looks unused, am I missing something?

As I explained before it's used from the tests modified in r145849.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.