Created attachment 403768 [details] [diff] [review] Patch v1 From Bug 495153 comment 3: >Not related to this patch, but why does this function return |false| in the >no-selection-controller case, but |textFound| in the no-document case? Seems >like it should return |textFound| in both cases. Worth a new bug? Also, kills some trailing whitespace, and doesn't take so many LOC to create the search ranges for highlighting.
Attachment #403768 - Flags: review?(gavin.sharp)
Attachment #403768 - Flags: review?(gavin.sharp) → review+
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a1
You need to log in before you can comment on or make changes to this bug.