WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 80136
[EFL][DRT] Implement LayoutTestController's methods related with editing
https://bugs.webkit.org/show_bug.cgi?id=80136
Summary
[EFL][DRT] Implement LayoutTestController's methods related with editing
Grzegorz Czajkowski
Reported
2012-03-02 02:52:22 PST
Implements EFL's LayoutTestController methods: - setSmartInsertDeleteEnabled, - setSelectTrailingWhitespaceEnabled - hasSpellingMarker. I am working on this implementation, patch will be uploaded soon.
Attachments
proposed patch
(14.66 KB, patch)
2012-03-05 07:00 PST
,
Grzegorz Czajkowski
no flags
Details
Formatted Diff
Diff
updated patch according to Kubo's review
(12.86 KB, patch)
2012-03-07 00:58 PST
,
Grzegorz Czajkowski
no flags
Details
Formatted Diff
Diff
rebased patch to HEAD
(12.83 KB, patch)
2012-03-07 01:37 PST
,
Grzegorz Czajkowski
rniwa
: review-
rniwa
: commit-queue-
Details
Formatted Diff
Diff
updated patch
(11.20 KB, patch)
2012-03-22 07:16 PDT
,
Grzegorz Czajkowski
no flags
Details
Formatted Diff
Diff
Show Obsolete
(3)
View All
Add attachment
proposed patch, testcase, etc.
Grzegorz Czajkowski
Comment 1
2012-03-05 07:00:42 PST
Created
attachment 130122
[details]
proposed patch
Raphael Kubo da Costa (:rakuco)
Comment 2
2012-03-05 08:18:39 PST
Comment on
attachment 130122
[details]
proposed patch View in context:
https://bugs.webkit.org/attachment.cgi?id=130122&action=review
I'd rather avoid doing what the Qt port does/used to do and make WebCoreSupport aware of DRT; the current code was written with that in mind, and I think we can keep doing so. Isn't it possible to make DRTSupportEefl::setSelectTrailingWhitespaceEnabled set some flag in EditClient itself or in ewk_view?
> LayoutTests/platform/efl/Skipped:67 > +editing/selection/doubleclick-inline-first-last-contenteditable.html
Please keep the list alphabetically sorted.
Grzegorz Czajkowski
Comment 3
2012-03-07 00:58:13 PST
Created
attachment 130563
[details]
updated patch according to Kubo's review
Grzegorz Czajkowski
Comment 4
2012-03-07 01:00:14 PST
(In reply to
comment #2
)
> (From update of
attachment 130122
[details]
) > View in context:
https://bugs.webkit.org/attachment.cgi?id=130122&action=review
> > I'd rather avoid doing what the Qt port does/used to do and make WebCoreSupport aware of DRT; the current code was written with that in mind, and I think we can keep doing so. Isn't it possible to make DRTSupportEefl::setSelectTrailingWhitespaceEnabled set some flag in EditClient itself or in ewk_view?
Ok, I added flag to EditorClinetEfl to set/get selectTrailingWhitespaceEnabled (similar to smartInsertDeleteEnabled).
> > > LayoutTests/platform/efl/Skipped:67 > > +editing/selection/doubleclick-inline-first-last-contenteditable.html > > Please keep the list alphabetically sorted.
Done.
Grzegorz Czajkowski
Comment 5
2012-03-07 01:37:02 PST
Created
attachment 130568
[details]
rebased patch to HEAD
Raphael Kubo da Costa (:rakuco)
Comment 6
2012-03-07 03:56:55 PST
Comment on
attachment 130568
[details]
rebased patch to HEAD Looks fine to me.
Ryosuke Niwa
Comment 7
2012-03-08 00:06:24 PST
Comment on
attachment 130568
[details]
rebased patch to HEAD View in context:
https://bugs.webkit.org/attachment.cgi?id=130568&action=review
> Source/WebKit/efl/WebCoreSupport/EditorClientEfl.cpp:235 > + m_smartInsertDeleteEnabled = enabled;
You also need to set m_selectTrailingWhitespaceEnabled false if enabled was true here.
> Source/WebKit/efl/WebCoreSupport/EditorClientEfl.cpp:245 > + m_selectTrailingWhitespaceEnabled = enabled;
You need to set m_smartInsertDeleteEnabled false if enabled was true here.
Grzegorz Czajkowski
Comment 8
2012-03-22 07:16:42 PDT
Created
attachment 133254
[details]
updated patch Added Ryosuke's suggestions. Removed hasSpellingMarker implementation from current patch - hasSpellingMarker should be used through Internals interface
https://bugs.webkit.org/show_bug.cgi?id=81300
. Patch for landing :)
WebKit Review Bot
Comment 9
2012-03-23 00:25:43 PDT
Comment on
attachment 133254
[details]
updated patch Clearing flags on attachment: 133254 Committed
r111835
: <
http://trac.webkit.org/changeset/111835
>
WebKit Review Bot
Comment 10
2012-03-23 00:25:48 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug