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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 1513143006: Add <pre> to nesting tags (Closed)

Created:
5 years ago by wychen
Modified:
5 years ago
Reviewers:
mdjones
Base URL:
git@github.com:chromium/dom-distiller.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Add <pre> to nesting tags If <pre> is not preserved, snippets like the following would break: <pre><code>blah</code></pre> BUG= R=mdjones@chromium.org Committed: 8fa737be250b8515f32dbca88f9f98afb1f10eb4

Patch Set 1 #

Patch Set 2 : add test #

Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -27 lines) Patch
M java/org/chromium/distiller/webdocument/WebTag.java View 1 chunk +1 line, -0 lines 0 comments Download
M javatests/org/chromium/distiller/ContentExtractorTest.java View 1 1 chunk +11 lines, -0 lines 0 comments Download
M javatests/org/chromium/distiller/webdocument/WebTagTest.java View 2 chunks +8 lines, -27 lines 0 comments Download

Messages

Total messages: 7 (4 generated)
wychen
PTAL
5 years ago (2015-12-11 02:49:02 UTC) #4
mdjones
lgtm
5 years ago (2015-12-11 16:59:57 UTC) #5
wychen
5 years ago (2015-12-16 03:00:16 UTC) #7
Message was sent while issue was closed. 
Committed patchset #2 (id:40001) manually as
8fa737be250b8515f32dbca88f9f98afb1f10eb4 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698