New issue
Advanced search Search tips

Issue 617701 link

Starred by 1 user
Status: Verified
Owner:
wychen@chromium.org
Closed: Jun 2016
Cc:
mdjones@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Start up latency of DOM distiller viewer should be recorded

Project Member Reported by wychen@chromium.org, Jun 6 2016 
Version: M52
OS: Android

The subjective latency can be defined as the click-to-render latency. Starting the viewer requires quite some time, and this could be in the critical path. We should record the start up latency of the viewer, and decide what to do next.

Comment 2 by wychen@chromium.org, Jun 10 2016

Labels: Merge-Request-52
Status: Verified (was: Untriaged)

Comment 3 by tin...@google.com, Jun 10 2016

Labels: -Merge-Request-52 Merge-Approved-52 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M52 (branch: 2743)
Project Member

Comment 4 by bugdroid1@chromium.org, Jun 10 2016

Labels: -merge-approved-52 merge-merged-2743
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/22746500f240c028a4ce65ee15e3d79dc9115051

commit 22746500f240c028a4ce65ee15e3d79dc9115051
Author: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Date: Fri Jun 10 21:24:27 2016

Record start up latency of DOM distiller viewer

BUG= 617701 

Review-Url: https://codereview.chromium.org/2011213002
Cr-Commit-Position: refs/heads/master@{#398115}
(cherry picked from commit b744ad3dc31c3d441687e9b99fac3eee22e90e32)

Review URL: https://codereview.chromium.org/2060583002 .

Cr-Commit-Position: refs/branch-heads/2743@{#319}
Cr-Branched-From: 2b3ae3b8090361f8af5a611712fc1a5ab2de53cb-refs/heads/master@{#394939}

[modify] https://crrev.com/22746500f240c028a4ce65ee15e3d79dc9115051/chrome/browser/dom_distiller/tab_utils.cc
[modify] https://crrev.com/22746500f240c028a4ce65ee15e3d79dc9115051/components/dom_distiller/content/browser/dom_distiller_viewer_source.cc
[modify] https://crrev.com/22746500f240c028a4ce65ee15e3d79dc9115051/components/dom_distiller/core/url_constants.cc
[modify] https://crrev.com/22746500f240c028a4ce65ee15e3d79dc9115051/components/dom_distiller/core/url_constants.h
[modify] https://crrev.com/22746500f240c028a4ce65ee15e3d79dc9115051/components/dom_distiller/core/url_utils.cc
[modify] https://crrev.com/22746500f240c028a4ce65ee15e3d79dc9115051/components/dom_distiller/core/url_utils.h
[modify] https://crrev.com/22746500f240c028a4ce65ee15e3d79dc9115051/components/dom_distiller/core/url_utils_unittest.cc
[modify] https://crrev.com/22746500f240c028a4ce65ee15e3d79dc9115051/tools/metrics/histograms/histograms.xml
Project Member

Comment 5 by bugdroid1@chromium.org, Jun 15 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/22746500f240c028a4ce65ee15e3d79dc9115051

commit 22746500f240c028a4ce65ee15e3d79dc9115051
Author: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Date: Fri Jun 10 21:24:27 2016

Record start up latency of DOM distiller viewer

BUG= 617701 

Review-Url: https://codereview.chromium.org/2011213002
Cr-Commit-Position: refs/heads/master@{#398115}
(cherry picked from commit b744ad3dc31c3d441687e9b99fac3eee22e90e32)

Review URL: https://codereview.chromium.org/2060583002 .

Cr-Commit-Position: refs/branch-heads/2743@{#319}
Cr-Branched-From: 2b3ae3b8090361f8af5a611712fc1a5ab2de53cb-refs/heads/master@{#394939}

[modify] https://crrev.com/22746500f240c028a4ce65ee15e3d79dc9115051/chrome/browser/dom_distiller/tab_utils.cc
[modify] https://crrev.com/22746500f240c028a4ce65ee15e3d79dc9115051/components/dom_distiller/content/browser/dom_distiller_viewer_source.cc
[modify] https://crrev.com/22746500f240c028a4ce65ee15e3d79dc9115051/components/dom_distiller/core/url_constants.cc
[modify] https://crrev.com/22746500f240c028a4ce65ee15e3d79dc9115051/components/dom_distiller/core/url_constants.h
[modify] https://crrev.com/22746500f240c028a4ce65ee15e3d79dc9115051/components/dom_distiller/core/url_utils.cc
[modify] https://crrev.com/22746500f240c028a4ce65ee15e3d79dc9115051/components/dom_distiller/core/url_utils.h
[modify] https://crrev.com/22746500f240c028a4ce65ee15e3d79dc9115051/components/dom_distiller/core/url_utils_unittest.cc
[modify] https://crrev.com/22746500f240c028a4ce65ee15e3d79dc9115051/tools/metrics/histograms/histograms.xml
►

Sign in to add a comment