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

Issue 1903853002: Stop fetching the next page if the first page has no content (Closed)

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

Description

Stop fetching the next page if the first page has no content In DOM distiller, if the distilled content of the first page is empty, it's very likely that the page is not good for distillation, so fetching the next page doesn't make sense. BUG=602139 Review URL: https://codereview.chromium.org/1891103002 Cr-Commit-Position: refs/heads/master@{#387759} (cherry picked from commit 6d72911b0cb47f5519288325fc502470815b9e7a)

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+178 lines, -111 lines) Patch
M components/dom_distiller/core/distiller.cc View 1 chunk +117 lines, -111 lines 0 comments Download
M components/dom_distiller/core/distiller_unittest.cc View 1 chunk +61 lines, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
wychen
4 years, 8 months ago (2016-04-20 19:22:35 UTC) #1
Message was sent while issue was closed.
Committed patchset #1 (id:1) to pending queue manually as
04991e720249701a0020a334a554f4c614eab338.

Powered by Google App Engine
This is Rietveld 408576698