[Webkit-unassigned] [Bug 16222] [GTK] Implement inline search and highlighting of matching strings.

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=16222


mrowe at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #17654|review+                     |
               Flag|                            |




------- Comment #10 from mrowe at apple.com  2007-12-05 08:18 PDT -------
(From update of attachment 17654)
The use of so many boolean parameters on Page::findString makes me
uncomfortable.  It will make the call sites hard to interpret without referring
to the headers.

The argument names in the header need to be named as their meanings are not at
all obvious based on the context.

I think the !!s should be removed as they were a Windows-ism.

The "found" variable in Page::findString looks to be redundant and could be
removed with a tiny bit of tweaking, without losing any clarity.

The ternary operator in Page::markAllMatchesForText could do with a little bit
of cleanup -- the spacing looks non-standard.


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