Page MenuHomePhabricator

Update html5 package from upstream.
Closed, ResolvedPublic


We should resync our forked html5 parser package with upstream ( We've made changes of two types:

  1. changes to allow us to pass in our tokens directly
  2. changes to support using domino instead of jsdom.

2b) changes to support using DOM4 instead of DOM3 (because domino only supports DOM4 methods)

It looks like upstream has landed changes to support DOM4 now, so the changes needed upstream to support domino might be very trivial. has to be changed to allow using domino instead of jsdom, based on options (see domino also doesn't support direct array access to attributes, so a few .item() accessor methods might need to be added, similar to

Version: unspecified
Severity: normal



Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 1:44 AM
bzimport added a project: Parsoid-DOM.
bzimport set Reference to bz47323.

Gwicke mentions that we have some foster parenting bugs, so hopefully resyncing with upstream will fix those. Upstream also has some issues with node 0.10 (; hopefully we'll inherit fixes for those (depending on when we resync).

See also which is a console.log we'd like to turn off upstream. is just the default document which does not necessarily need to be changed. Passing in a domino document should work fine and will avoid using jsdom.

Change 79442 had a related patch set uploaded by Arlolra:
WIP: Update html5 package from upstream.

Change 79442 merged by jenkins-bot:
Update html5 package from upstream.