Bug 128380 - [GTK] Minibrowser: Matched words of a searched text are not highlighted after reopening search bar
Summary: [GTK] Minibrowser: Matched words of a searched text are not highlighted after...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Minor
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-02-07 09:53 PST by Diego Pino
Modified: 2014-02-08 00:22 PST (History)
5 users (show)

See Also:


Attachments
Patch (3.61 KB, patch)
2014-02-07 09:59 PST, Diego Pino
no flags Details | Formatted Diff | Diff
Patch (1.64 KB, patch)
2014-02-07 10:35 PST, Diego Pino
no flags Details | Formatted Diff | Diff
Patch (1.65 KB, patch)
2014-02-07 12:50 PST, Diego Pino
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Diego Pino 2014-02-07 09:53:01 PST
In Minibrowser, when searching for some text in a page, all matches are highlighted. Clicking on the Up and Down buttons in the search bar, navigates through the list of matched words. When the search bar is closed down, all the marked words are unhighlighted. However, when the search bar is reopened, although the search bar still contains the results of the latest search (and in fact, it is possible to navigate through the list of matched words using the Up and Down buttons), the words are not highlighted.

Steps to reproduce:

1. Open a the Minibrowser with the default URL (http://www.webkitgtk.org/) for instance.
2. Press Ctrl+F to open the search bar and type "the".
3. All the "the" words will be highlighted. Navigate through the list of matched words.
4. Close the search bar.
5. Press Ctrl+F to open the search bar again.
6. The search bar contains the results of the last search. It's possible to navigate through the list of matches but there are no highlighted words.

Expected result: When opening the search bar again, highlight the matched words of the last search. Notice that when reopening the search bar, its edit box is not empty but contains the last searched text.
Comment 1 Diego Pino 2014-02-07 09:59:45 PST
Created attachment 223462 [details]
Patch
Comment 2 WebKit Commit Bot 2014-02-07 10:00:26 PST
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API
Comment 3 Carlos Garcia Campos 2014-02-07 10:05:44 PST
I would be great if we could add a unit tests for this case.
Comment 4 Diego Pino 2014-02-07 10:35:50 PST
Created attachment 223471 [details]
Patch
Comment 5 Diego Pino 2014-02-07 12:50:15 PST
Created attachment 223484 [details]
Patch
Comment 6 WebKit Commit Bot 2014-02-08 00:22:05 PST
Comment on attachment 223484 [details]
Patch

Clearing flags on attachment: 223484

Committed r163699: <http://trac.webkit.org/changeset/163699>
Comment 7 WebKit Commit Bot 2014-02-08 00:22:08 PST
All reviewed patches have been landed.  Closing bug.