Issue 2278503002: [scheduler] Fix trace counter for renderer main thread load.

Issue 2278503002: [scheduler] Fix trace counter for renderer main thread load. (Closed)

Created:
4 years, 3 months ago by altimin

Modified:
4 years, 3 months ago

Reviewers:
Sami, alex clarke (OOO till 29th)

CC:
chromium-reviews, blink-reviews, scheduler-bugs_chromium.org

Base URL:
https://chromium.googlesource.com/chromium/src.git@master

Target Ref:
refs/pending/heads/master

Project:
chromium

Visibility:
Public.

[scheduler] Fix trace counter for renderer main thread load. Convert load level from fraction value from [0, 1] range to 0-100% percentage for using as trace counter. R=alexclarke@chromium.org, skyostil@chromium.org BUG=639852 Committed: https://crrev.com/b437716023a23fc12ad9fff36c88de8034d8e796 Cr-Commit-Position: refs/heads/master@{#414057}

Created: 4 years, 3 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+6 lines, -4 lines)			Patch
	M	third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.cc	View		1 chunk	+6 lines, -4 lines	0 comments	Download

Total messages: 10 (5 generated)

Sami

lgtm (nit: instead of "fix" please say what actually changed.)

4 years, 3 months ago (2016-08-24 11:29:16 UTC) #2

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2278503002/1

4 years, 3 months ago (2016-08-24 11:31:10 UTC) #6

commit-bot: I haz the power

4 years, 3 months ago (2016-08-24 13:00:49 UTC) #10

Message was sent while issue was closed.

Patchset 1 (id:??) landed as
https://crrev.com/b437716023a23fc12ad9fff36c88de8034d8e796
Cr-Commit-Position: refs/heads/master@{#414057}