[Webkit-unassigned] [Bug 6616] Double-clicking on a search result seems broken

[bugzilla-daemon at opendarwin.org]

http://bugzilla.opendarwin.org/show_bug.cgi?id=6616


darin at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #6830|review?                     |review+
               Flag|                            |




------- Comment #2 from darin at apple.com  2006-03-04 11:56 PDT -------
(From update of attachment 6830)
The finalize method needs to call [super finalize]. But also, as long as
there's an observer installed, the object will never get finalized because the
pointers in the notification center will keep it alive. You need to come up
with some other way to handle the object lifetime. Some way of "shutting down"
other than actually deallocating the object that will work both with and
without GC. If this is a global object that's never deallocated, then you just
can not worry about it.

setSearchQuery: and searchPerformed: should use copy on the passed-in string
instead of retain.

+    BOOL selectFirst = (![_private->searchResults count] ? YES : NO);

No need for ? YES : NO on something that's already a boolean.

I noticed that for JavaScript you aren't putting the start brace for functions
on a separate line. I think you should match our C coding style more closely
there.

+    } else if(!query.length && searchActive) {

Above line is missing a space after the if.

+        <input id="search" type="search" autosave="nodeSearch" results="20"
placeholder="Search" incremental="incremental"
onsearch="performSearch(this.value)" />

Not sure why you are using self-closing tags in this file. Is it XHTML or HTML?

Generally looks good. I'm going to mark it review+ even though the finalize
method is wrong.


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