Closed Bug 1343991 Opened 9 years ago Closed 9 years ago

Update PageMetadata.jsm for Activity Stream's needs

Categories

(Firefox Graveyard :: Activity Streams: General, enhancement)

Product:
Firefox Graveyard
Component:
Activity Streams: General
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: jlockhart, Assigned: jlockhart)

Details

In Activity Stream, we use the following module for parsing page metadata: https://github.com/mozilla/page-metadata-parser This includes lists of relevant DOM tags for things like high res icons, preview images, content types, etc as well as the logic to iteratively extract them from the document. In mozilla central, there is the PageMetadata.jsm module: https://dxr.mozilla.org/mozilla-central/source/toolkit/modules/PageMetadata.jsm which performs many similar tasks but does not include all of the data that we use in Activity Stream. We can update PageMetadata.jsm to include all of the types of data we need by adding portions of our parser to this module.
Assignee: nobody → jkerim
We have shifted from this approach towards handling metadata directly in ContentLinkHandler, see here: https://bugzilla.mozilla.org/show_bug.cgi?id=1352459
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.