Closed Bug 776792 Opened 12 years ago Closed 12 years ago

Fire an Activity for application/pdf document

Categories

(Firefox OS Graveyard :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
DeveloperPhone

People

(Reporter: vingtetun, Assigned: vingtetun)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch Patch (obsolete) — Splinter Review
Attachment #645165 - Flags: review?(jones.chris.g)
Attached patch PatchSplinter Review
Attachment #645165 - Attachment is obsolete: true
Attachment #645165 - Flags: review?(jones.chris.g)
Attachment #645166 - Flags: review?(jones.chris.g)
Attachment #645166 - Flags: review?(jones.chris.g) → review+
It makes me sad that this code is locked away in b2g/.  How can we start making this part of gecko proper?
Comment on attachment 645166 [details] [diff] [review]
Patch

Review of attachment 645166 [details] [diff] [review]:
-----------------------------------------------------------------

The PDFContentHandler.js file is missing from the patch ;)

Also, since we could use the same content handler for different content types, we could rename it and make it send a $MIME-handler observer notification
That would be nice. We want this for doc and xls etc as well.
https://hg.mozilla.org/integration/mozilla-inbound/rev/511943aecd60

Fabrice, I have added the file and called it ContentHandler.js :)
Target Milestone: --- → DeveloperPhone
https://hg.mozilla.org/mozilla-central/rev/511943aecd60
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Is there a bug tracking this for other document types?
You need to log in before you can comment on or make changes to this bug.