Skip to content

Deploy 20181119#1508

Merged
julienw merged 37 commits into
productionfrom
master
Nov 19, 2018
Merged

Deploy 20181119#1508
julienw merged 37 commits into
productionfrom
master

Conversation

@julienw

@julienw julienw commented Nov 19, 2018

Copy link
Copy Markdown
Contributor
  • documents how to use linux perf
  • better naming for parent and content processes
  • locale support for number formatting
  • fix a problem with the responsiveness handling
  • adds a sidebar to the marker table
  • fixes problems with the line and column numbers

jesup and others added 30 commits November 6, 2018 13:05
Rename 'Main Thread' to 'Parent Process', and add 'Process' to 'Content' and 'Plugin' (#1429)
Then these functions are used in a lot of places, especially in the call tree and in tooltips.

Fixes #1322
…Fixes #1472)

The chrome import script does not provide responsiveness values, and the
type was made optional. Previously the getJankInstances had coerced null
values into numeric. This patch corrects the usage to just skip null
values. It turns out that occasionally the sampler skips a
responsiveness value. Inspecting a profile with this behavior revealed
that it was best to just ignore those samples. In addition, the function
still works for completely null or undefined values.

Tests were also added to this patch explicitely test this behavior.
Handle optional profile.thread[i].processName (Fixes #1451)
Add 'pages' array to processed profile format and remove the URLs while sharing without URLs
@julienw julienw merged commit 6600ac0 into production Nov 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants