Update PageMetadata.jsm for Activity Stream's needs

RESOLVED WONTFIX

Status

()

Firefox
Activity Streams: General
RESOLVED WONTFIX
3 months ago
a month ago

People

(Reporter: jkerim, Assigned: jkerim)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

3 months ago
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)

Updated

3 months ago
Assignee: nobody → jkerim
(Assignee)

Comment 1

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