Bug 68547 - [WK2][GTK]App changes - Implementation of TextChecker
Summary: [WK2][GTK]App changes - Implementation of TextChecker
Status: RESOLVED DUPLICATE of bug 90268
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on: 68532 68548
Blocks:
  Show dependency treegraph
 
Reported: 2011-09-21 11:10 PDT by Ravi Phaneendra Kasibhatla
Modified: 2013-01-04 00:45 PST (History)
5 users (show)

See Also:


Attachments
App changes for TextChecker (12.28 KB, patch)
2011-09-21 11:10 PDT, Ravi Phaneendra Kasibhatla
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ravi Phaneendra Kasibhatla 2011-09-21 11:10:32 PDT
Created attachment 108185 [details]
App changes for TextChecker 

The bug captures the app changes required for verifying the TextChecker feature for WebKit2 GTK port.
Comment 1 WebKit Review Bot 2011-09-21 11:13:30 PDT
Attachment 108185 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Tools/ChangeLog', u'Tools/MiniBrowser/gtk/..." exit_code: 1

Last 3072 characters of output:
claration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:738:  Declaration has space between * and variable name in EnchantDict* dict  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:750:  Declaration has space between * and variable name in BrowserWindow* window  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:751:  Declaration has space between * and variable name in GSList* dicts  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:756:  Declaration has space between * and variable name in char* string  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:758:  Declaration has space between * and variable name in PangoLanguage* language  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:759:  Declaration has space between * and variable name in PangoLogAttr* attrs  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:782:  Declaration has space between * and variable name in gchar* cstart  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:784:  Declaration has space between * and variable name in gchar* word  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:789:  Declaration has space between * and variable name in EnchantDict* dict  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:813:  Declaration has space between * and variable name in BrowserWindow* window  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:814:  Declaration has space between * and variable name in GSList* dicts  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:820:  Declaration has space between * and variable name in char* string  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:827:  Declaration has space between * and variable name in EnchantDict* dict  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:852:  Declaration has space between * and variable name in BrowserWindow* window  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:853:  Declaration has space between * and variable name in GSList* dicts  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:858:  Declaration has space between * and variable name in char* string  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:862:  Declaration has space between * and variable name in EnchantDict* dict  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:872:  Declaration has space between * and variable name in BrowserWindow* window  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:873:  Declaration has space between * and variable name in GSList* dicts  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:878:  Declaration has space between * and variable name in char* string  [whitespace/declaration] [3]
Tools/MiniBrowser/gtk/BrowserWindow.c:882:  Declaration has space between * and variable name in EnchantDict* dict  [whitespace/declaration] [3]
Total errors found: 32 in 3 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 2 Gustavo Noronha (kov) 2011-09-21 11:18:22 PDT
Comment on attachment 108185 [details]
App changes for TextChecker 

Attachment 108185 [details] did not pass gtk-ews (gtk):
Output: http://queues.webkit.org/results/9765853
Comment 3 Martin Robinson 2012-12-11 01:20:38 PST
I think perhaps this is already implemented with API backing it. :(

*** This bug has been marked as a duplicate of bug 90268 ***
Comment 4 Eric Seidel (no email) 2013-01-04 00:45:50 PST
Comment on attachment 108185 [details]
App changes for TextChecker 

Cleared review? from attachment 108185 [details] so that this bug does not appear in http://webkit.org/pending-review.  If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review again).