Bug 101570 - [EFL] Add a method to the TextCheckerEnchant class to check whether any dictionary is loaded
Summary: [EFL] Add a method to the TextCheckerEnchant class to check whether any dicti...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Grzegorz Czajkowski
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-08 02:47 PST by Grzegorz Czajkowski
Modified: 2012-11-08 23:36 PST (History)
7 users (show)

See Also:


Attachments
proposed patch (6.21 KB, patch)
2012-11-08 02:49 PST, Grzegorz Czajkowski
gustavo: review-
Details | Formatted Diff | Diff
apply Gustavo's suggestions (6.21 KB, patch)
2012-11-08 06:04 PST, Grzegorz Czajkowski
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Grzegorz Czajkowski 2012-11-08 02:47:25 PST
Expose a new method to check whether a vector of loaded languages is empty.
WK2-EFL needs it to set the default language (if the client didn't set any) when spelling setting is being enabled. This change allows to do not get vector of dictionaries to just check its size.
Comment 1 Grzegorz Czajkowski 2012-11-08 02:49:50 PST
Created attachment 172972 [details]
proposed patch
Comment 2 Gustavo Noronha (kov) 2012-11-08 05:41:23 PST
Comment on attachment 172972 [details]
proposed patch

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

Mostly a readability/hackability change, so I'll be nit-picky and say languagesLoaded sounds to me like it would return the languages that have been loaded. Also, I'd prefer maintaining the "dictionary" terminology, so perhaps hasDictionary?

> Source/WebCore/ChangeLog:11
> +        spelling setting is being enabled. This change allows to do not get
> +        vector of dictionaries to just check its size.

This last sentence is a bit broken. How about "This change makes it more convenient to check whether the dictionaries vector is empty."?
Comment 3 Grzegorz Czajkowski 2012-11-08 06:04:38 PST
Created attachment 173027 [details]
apply Gustavo's suggestions
Comment 4 WebKit Review Bot 2012-11-08 23:36:41 PST
Comment on attachment 173027 [details]
apply Gustavo's suggestions

Clearing flags on attachment: 173027

Committed r134021: <http://trac.webkit.org/changeset/134021>
Comment 5 WebKit Review Bot 2012-11-08 23:36:45 PST
All reviewed patches have been landed.  Closing bug.