New issue
Advanced search Search tips

Issue 815244 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue pdfium:1019



Sign in to add a comment

Feature: Add UMA bridge between chrome and PDFium

Project Member Reported by tsepez@chromium.org, Feb 23 2018

Issue description

PDFium itself doesn't have any ability to report UMA-like data back to its embedder.  Consider adding a generic framework to allow this to happen while still maintiaing an embedder-agnostic API.
 

Comment 1 by tsepez@chromium.org, Feb 23 2018

Blocking: pdfium:1019

Comment 2 by thestig@google.com, Feb 23 2018

Components: Internals>Plugins>PDF
Status: Started (was: Untriaged)
Sorry for not giving input earlier. Are we set on having a UMA bridge? If it's generally useful, (probably yes) then let's keep going. But if it is the case that we only want the JS stats, an alternative could be to just extend IPDF_JSPLATFORM.
The IPDF_JSPLATFORM is setup as part of the formfill environment which happens after the document is loaded. I believe you can have JS run on document load, so we'd potentially miss JS firing.

We could possibly do it in the data we pass to FPDF_LIBRARY_CONFIG at library configuration time.
I think having the UMA bridge is good. I'd like to see us add some performance metrics like how long does document load take, or page rendering time.
Owner: ----
Status: Untriaged (was: Started)
Setting PDF bugs assigned to me back to untriaged so they can get re-assigned as needed.
I started working on this before other work came up:

PDFium CL: https://pdfium-review.googlesource.com/c/pdfium/+/27690
Chromium CL: https://chromium-review.googlesource.com/c/chromium/src/+/949665
Owner: thestig@chromium.org
Status: Assigned (was: Untriaged)
Assigning to thestig@ for evaluating if we still want to do this.

Sign in to add a comment