Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(120)

Issue 1908173005: Roll DOM Distiller JavaScript distribution package (Closed)

Created:
4 years, 8 months ago by mdjones
Modified:
4 years, 8 months ago
Reviewers:
wychen
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Roll DOM Distiller JavaScript distribution package Diff since last roll: https://github.com/chromium/dom-distiller/compare/349b932fc2...c3edb386a9 Picked up changes: c3edb38 Add travis-ci support adbdbec Fix for Schema.org title extraction. f40e521 Lazy evaluation of MarkupParsers 84cfcd2 Retain image sizes 3b3253f Make long strings easier to compare in assertion failure fc19f02 Clean attributes of mRoot between tests properly ccefff9 Handle drop cap letters 9101ca4 Fix spelling in comments cfb7588 Update README.md 4d0809f Reformat README.md to Google style 72998ef Add scripts for distillability modeling 7310b0f Process the root element in various DomUtil functions BUG=483172, 552408, 576553, 589294, 591143, 593128, 604177, 605484 Committed: https://crrev.com/04fe15d5c151bd41ad3f757e03977218f78e1828 Cr-Commit-Position: refs/heads/master@{#389270}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download
M third_party/dom_distiller_js/README.chromium View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 10 (5 generated)
mdjones
ptal
4 years, 8 months ago (2016-04-22 18:40:27 UTC) #2
wychen
lgtm
4 years, 8 months ago (2016-04-22 19:06:56 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1908173005/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1908173005/1
4 years, 8 months ago (2016-04-22 19:58:46 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 8 months ago (2016-04-22 22:26:35 UTC) #8
commit-bot: I haz the power
4 years, 8 months ago (2016-04-22 22:28:14 UTC) #10
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/04fe15d5c151bd41ad3f757e03977218f78e1828
Cr-Commit-Position: refs/heads/master@{#389270}

Powered by Google App Engine
This is Rietveld 408576698