Closed Bug 1343991 Opened 4 years ago Closed 3 years ago

Update PageMetadata.jsm for Activity Stream's needs

Categories

(Firefox :: Activity Streams: General, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: jkerim, Assigned: jkerim)

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: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.