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

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

Created:
3 years, 8 months ago by wychen
Modified:
3 years, 8 months ago
Reviewers:
mdjones
CC:
chromium-reviews, Olivier, noyau (Ping after 24h)
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Roll DOM Distiller JavaScript distribution package Diff since last roll: https://github.com/chromium/dom-distiller/compare/489c6609cb...8de0cacfed Picked up changes: 8de0cac Handle image lazy loading on Wikipedia 9c2021c Ignore the "Edit" links in wiki pages cd538d1 Unhide folded sections in wiki pages 4d66ebb Make sure images are loaded before running tests 2d7b629 Make sure lazily-loaded images have absolute URL BUG=647667 Review-Url: https://codereview.chromium.org/2809883002 Cr-Commit-Position: refs/heads/master@{#463503} Committed: https://chromium.googlesource.com/chromium/src/+/e7dcd5dedea72f0619d7523bbca7344794f677b2

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: 8 (4 generated)
wychen
PTAL and land it. Thanks!
3 years, 8 months ago (2017-04-11 00:57:50 UTC) #2
mdjones
lgtm
3 years, 8 months ago (2017-04-11 01:21:06 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2809883002/1
3 years, 8 months ago (2017-04-11 01:22:09 UTC) #5
commit-bot: I haz the power
3 years, 8 months ago (2017-04-11 02:46:42 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/e7dcd5dedea72f0619d7523bbca7...

Powered by Google App Engine
This is Rietveld 408576698