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

Issue 1809473003: Decrease mismatches in feature extraction (Closed)

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

Description

Decrease mismatches in feature extraction Fix isVisible() to match the behavior of native implementation in Chrome https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit/Source/core/dom/DocumentStatisticsCollector.cpp&l=58&rcl=1457954878 BUG=595120 R=mdjones@chromium.org Committed: 8a12e18c8c74a9c537cd994405405a9e6e98cd99

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M heuristics/distillable/extract_features.js View 1 chunk +1 line, -1 line 0 comments Download

Dependent Patchsets:

Messages

Total messages: 5 (2 generated)
wychen
PTAL
4 years, 9 months ago (2016-03-15 22:10:21 UTC) #2
mdjones
lgtm
4 years, 9 months ago (2016-03-16 15:54:45 UTC) #3
wychen
4 years, 4 months ago (2016-08-10 01:36:51 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
8a12e18c8c74a9c537cd994405405a9e6e98cd99 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698