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

Issue 884323002: Roll DomDistillerJS (Closed)

Created:
5 years, 10 months ago by cjhopman
Modified:
5 years, 10 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 DomDistillerJS Picked up changes: 197d576 Delete a bunch of gwt test environment-specific things ae707d0 Remove DomToSaxVisitor and rename some classes e8b2fcd Add a WebDocument class 52c4790 Support verbose output in jstest ac006fe Remove ant target "test.dev" and "test.prod" 7db0bf4 Trim trailing spaces in java source code 0338d85 Fix all invalid URLs in tests 2489c81 Fix "Support test.filter for jstests" 442f8ee Reorganize all the code in the project. 2044b6f Support test.filter for jstests ddb81f8 Disable some tests in PagingLinksFinderTest 82f505c Move PagingLinksFinderTest from test/ to jstest/ 2f9ec7d Add a devtools panel to inject extraction script into the inspected window 9962247 Remove BoilerpipeProcessingException and other unused things. 491c7ce Use pre-compiled regexes for many things in PagingLinksFinder e26e5ff Fix "Move ElementActionTest and TextBlockBuilderTest from test/ to jstest/" 6893a1a Move ElementActionTest and TextBlockBuilderTest from test/ to jstest/ 6a97c63 Add util/TextBlockBuilderTest into DomDistillerTestSuite d331041 Fix ElementActionTest BUG=440102, 440977 Committed: https://crrev.com/5626a96f6c3aa46ef74edee24d8281dc9f02a2bc Cr-Commit-Position: refs/heads/master@{#313648}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+179 lines, -189 lines) Patch
M components/dom_distiller/core/javascript/domdistiller.js 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
M third_party/dom_distiller_js/package/js/domdistiller.js View 1 chunk +176 lines, -186 lines 0 comments Download
M third_party/dom_distiller_js/package/proto/dom_distiller.proto View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 7 (2 generated)
cjhopman
nyquist: *
5 years, 10 months ago (2015-01-29 00:44:31 UTC) #2
nyquist
lgtm
5 years, 10 months ago (2015-01-29 00:55:04 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/884323002/1
5 years, 10 months ago (2015-01-29 00:55:17 UTC) #5
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 10 months ago (2015-01-29 01:39:12 UTC) #6
commit-bot: I haz the power
5 years, 10 months ago (2015-01-29 02:10:01 UTC) #7
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/5626a96f6c3aa46ef74edee24d8281dc9f02a2bc
Cr-Commit-Position: refs/heads/master@{#313648}

Powered by Google App Engine
This is Rietveld 408576698