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

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

Created:
5 years, 6 months ago by wychen
Modified:
5 years, 6 months ago
Reviewers:
nyquist
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/aa79b394e7...6300dcfdb5 Picked up changes: 6300dcf Leverage semantic elements to find the main article 61e7226 Add Vagrant support bf64df9 Make TableClassifierTest independent of browser window dimention 7a10992 Fix word count issue for Chinese and Japanese 90e7f8c Bump versions of chromedriver and selenium f707680 Fix options "shuffle" and "no_sandbox" in run_jstests.py 640e399 Disable sandbox and run all the tests in continuous integration 869d139 Update Chrome if it is too old 11f6b8c Avoid chromedriver elsewhere if any 4f63157 Allow tests to run without Chrome sandbox edafdf1 Install Chrome when necessary 2c4e93b BR elements should be folded into WebText BUG=431067, 483149, 483710, 483713, 484750, 485177, 485829, 495254 Committed: https://crrev.com/e840735a47193caadc96ede969d30f7523f04e18 Cr-Commit-Position: refs/heads/master@{#333733}

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: 15 (6 generated)
wychen
PTAL
5 years, 6 months ago (2015-06-10 01:14:00 UTC) #2
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1174513005/1
5 years, 6 months ago (2015-06-10 01:16:13 UTC) #4
commit-bot: I haz the power
Dry run: No LGTM from a valid reviewer yet. Only full committers are accepted. Even ...
5 years, 6 months ago (2015-06-10 01:16:17 UTC) #6
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1174513005/1
5 years, 6 months ago (2015-06-10 04:28:43 UTC) #8
nyquist
lgtm
5 years, 6 months ago (2015-06-10 04:31:20 UTC) #9
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
5 years, 6 months ago (2015-06-10 06:20:06 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1174513005/1
5 years, 6 months ago (2015-06-10 14:57:04 UTC) #13
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 6 months ago (2015-06-10 15:00:01 UTC) #14
commit-bot: I haz the power
5 years, 6 months ago (2015-06-10 15:00:48 UTC) #15
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/e840735a47193caadc96ede969d30f7523f04e18
Cr-Commit-Position: refs/heads/master@{#333733}

Powered by Google App Engine
This is Rietveld 408576698