Closed Bug 580520 Opened 11 years ago Closed 11 years ago

Update Comment on mimeTypeIsTextBased to Include Request to Update findbar.xml Upon Any Changes

Categories

(SeaMonkey :: Find In Page, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.1a3

People

(Reporter: bfrisch, Assigned: bfrisch)

Details

Attachments

(1 file, 1 obsolete file)

Follow-up bug from bug 97023 comment 51 where Philip Chee wrote:
> In Firefox there is this comment:
> 
> /**
>  * Returns true if |aMimeType| is text-based, false otherwise.
>  *
>  * @param aMimeType
>  *        The MIME type to check.
>  *
>  * If adding types to this function, please also check the similar
>  * function in findbar.xml
>  */
> function mimeTypeIsTextBased(aMimeType)
> 
> Perhaps we should add a similar comment to our implementation of
> mimeTypeIsTextBased()
Attached patch Add the Comment v.1 (obsolete) — Splinter Review
This patch should add an appropriate comment.
Assignee: nobody → bfrisch
Status: NEW → ASSIGNED
Attachment #458911 - Flags: review?(neil)
Comment on attachment 458911 [details] [diff] [review]
Add the Comment v.1

>+ * If adding types to this function, please also check the similar
>+ * functions in suite/commom/bindings/findbar.xml and
suite/commom/bindings/findbar.xml just inherits the toolkit version, it doesn't have its own function.

>+ * mozilla/toolkit/widgets/findbar.xml.
Did you mean mozilla/toolkit/content/widgets/findbar.xml?
This fixed the wrong path to findbar.xml in the toolkit.  It also gets rid of the reference to the findbar.xml in suite, as now that I think about it, we probably shouldn't have a need to update the function in just the suite findbar.xml in the future.

Thanks.
Attachment #458911 - Attachment is obsolete: true
Attachment #459776 - Flags: review?(neil)
Attachment #458911 - Flags: review?(neil)
Attachment #459776 - Flags: review?(neil) → review+
Pushed changeset 12adef298038 to comm-central.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a3
You need to log in before you can comment on or make changes to this bug.