New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 798354 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

4.2%-5.6% regression in smoothness.tough_animation_cases at 525925:525993

Project Member Reported by alexclarke@chromium.org, Jan 2 2018

Issue description

See the link to graphs below.
 
Showing comments 1031 - 1130 of 1130 Older

Comment 1031 Deleted

Comment 1032 Deleted

Comment 1033 Deleted

Comment 1034 Deleted

Comment 1035 Deleted

Comment 1036 Deleted

Comment 1037 Deleted

Comment 1038 Deleted

Comment 1039 Deleted

Comment 1040 Deleted

Comment 1041 Deleted

Comment 1042 Deleted

Comment 1043 Deleted

Comment 1044 Deleted

Comment 1045 Deleted

Comment 1046 Deleted

Comment 1047 Deleted

Comment 1048 Deleted

Comment 1049 Deleted

Comment 1050 Deleted

Comment 1051 Deleted

Comment 1052 Deleted

Comment 1053 Deleted

Comment 1054 Deleted

Comment 1055 Deleted

Comment 1056 Deleted

Comment 1057 Deleted

Comment 1058 Deleted

Comment 1059 Deleted

Comment 1060 Deleted

Comment 1061 Deleted

Comment 1062 Deleted

Comment 1063 Deleted

Comment 1064 Deleted

Comment 1065 Deleted

Comment 1066 Deleted

Comment 1067 Deleted

Comment 1068 Deleted

Comment 1069 Deleted

Comment 1070 Deleted

Comment 1071 Deleted

Comment 1072 Deleted

Comment 1073 Deleted

Comment 1074 Deleted

Comment 1075 Deleted

Comment 1076 Deleted

Comment 1077 Deleted

Comment 1078 Deleted

Comment 1079 Deleted

Comment 1080 Deleted

Comment 1081 Deleted

Comment 1082 Deleted

Comment 1083 Deleted

Comment 1084 Deleted

Comment 1085 Deleted

Comment 1086 Deleted

Comment 1087 Deleted

Comment 1088 Deleted

Comment 1089 Deleted

Comment 1090 Deleted

Comment 1091 Deleted

Comment 1092 Deleted

Comment 1093 Deleted

Comment 1094 Deleted

Comment 1095 Deleted

Comment 1096 Deleted

Comment 1097 Deleted

Comment 1098 Deleted

Comment 1099 Deleted

Comment 1100 Deleted

Comment 1101 Deleted

Comment 1102 Deleted

Comment 1103 Deleted

Comment 1104 Deleted

Comment 1105 Deleted

Comment 1106 Deleted

Comment 1107 Deleted

Comment 1108 Deleted

Comment 1109 Deleted

Comment 1110 Deleted

Comment 1111 Deleted

Comment 1112 Deleted

Comment 1113 Deleted

Comment 1114 Deleted

Comment 1115 Deleted

Comment 1116 Deleted

Comment 1117 Deleted

Comment 1118 Deleted

I've deployed an emergency fix for the infinite bug comments:
https://chromium-review.googlesource.com/c/catapult/+/948266

And filed a bug:
https://github.com/catapult-project/catapult/issues/4307
Cc: dtu@chromium.org simonhatch@chromium.org
Kicking of a few more bisects.

+Dave, Simon any chance we could use the monorail API to get rid of all these comments?

Comment 1123 by dtu@chromium.org, Mar 16 2018

Done.
Cc: haraken@chromium.org yutak@chromium.org
Owner: yutak@chromium.org
Status: Assigned (was: Untriaged)
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/129cf8a1440000

Remove WebTaskRunner::PostTask() for CrossThreadClosure. by yutak@chromium.org
https://chromium.googlesource.com/chromium/src/+/ef414c066331d7648d32d21da3b81f06e5058458

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions
Owner: ----
Status: Untriaged (was: Assigned)
The CL just removes an unused function. There's no production code change, so
this bisect result doesn't really make sense.

Comment 1126 by dtu@chromium.org, Mar 16 2018

Cc: -haraken@chromium.org -yutak@chromium.org
Oh, yes, the reason why I hadn't re-run the bisect is because the partial results looked really noisy. frame_times is generally hard to bisect on.
Cc: rdevlin....@chromium.org mlippautz@chromium.org rdsmith@chromium.org xunji...@chromium.org kinuko@chromium.org haraken@chromium.org tsepez@chromium.org shimazu@chromium.org
Owner: rdsmith@chromium.org
Status: Assigned (was: Untriaged)
📍 Found significant differences after each of 3 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/15db5bd1440000

S13nServiceWorker: Call OnNoWork when all stream responses finish by shimazu@chromium.org
https://chromium.googlesource.com/chromium/src/+/cad7fcc1c6659e192ac78d5c4f7dd18e21d4dfae

[oilpan] HeapHashSet add support for incremental marking by mlippautz@chromium.org
https://chromium.googlesource.com/chromium/src/+/07a928267fc62ea5d747a2323dc0244546a70da7

Shift extension API code to using Mojo CookieManager interface. by rdsmith@chromium.org
https://chromium.googlesource.com/chromium/src/+/e93faff1da7b32ad86b9402ae20289921d622d24

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions
Owner: ----
Status: Untriaged (was: Assigned)
(-rdsmith@ who moved on to a different project)

re93faff1da7b32ad86b9402ae20289921d622d24 only changes the Extensions code. Perf impact should be minimal. I don't think smoothness.tough_animation_cases benchmark exercises the Extensions code paths.
cad7fcc1c6659e192ac78d5c4f7dd18e21d4dfae in c#1127 too. It only works when ServiceWorkerServicification flag is on.
Status: WontFix (was: Untriaged)
Most of these recovered, and can't find much more in the bisects to act on.
Showing comments 1031 - 1130 of 1130 Older

Sign in to add a comment