Closed
Bug 1412837
Opened 7 years ago
Closed 6 years ago
testbed
Categories
(Invalid Bugs :: General, enhancement)
Invalid Bugs
General
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: gasolin, Unassigned)
Details
Attachments
(1 file, 5 obsolete files)
59 bytes,
text/x-review-board-request
|
Details |
find a place for try test experiment
Comment hidden (mozreview-request) |
Reporter | ||
Comment 2•7 years ago
|
||
remove unused toolbar function https://treeherder.mozilla.org/#/jobs?repo=try&revision=dfc47e94333b30de7dfba55ad0e5edd079170363
Comment hidden (mozreview-request) |
Reporter | ||
Comment 4•7 years ago
|
||
with shouldComponentUpdate https://treeherder.mozilla.org/#/jobs?repo=try&revision=0fbf6b08858c
Comment hidden (mozreview-request) |
Reporter | ||
Updated•7 years ago
|
Attachment #8923385 -
Attachment is obsolete: true
Comment hidden (mozreview-request) |
Reporter | ||
Comment 7•7 years ago
|
||
clean shouldComponentUpdate perf looks good https://treeherder.mozilla.org/perf.html#/comparesubtest?originalProject=mozilla-central&newProject=try&newRevision=e9a892d92d11e303d47d0060f7b6209a126ab090&originalSignature=edaec66500db21d37602c99daa61ac983f21a6ac&newSignature=edaec66500db21d37602c99daa61ac983f21a6ac&filter=netmonitor&framework=1&selectedTimeRange=172800
Reporter | ||
Comment 8•7 years ago
|
||
based on above but removed searchbox ref https://treeherder.mozilla.org/#/jobs?repo=try&revision=b6d4165e5c07
Comment hidden (mozreview-request) |
Reporter | ||
Comment 10•7 years ago
|
||
Comment 8 does not perform well, main difference might caused by searchBox shouldComponentUpdate, update PR to test that
Comment hidden (mozreview-request) |
Reporter | ||
Comment 12•7 years ago
|
||
do shouldComponentUpdate in searchbox shows some gain https://treeherder.mozilla.org/perf.html#/comparesubtest?originalProject=mozilla-central&newProject=try&newRevision=48f7d0aace0d6776f95b2a6255205e30586406db&originalSignature=edaec66500db21d37602c99daa61ac983f21a6ac&newSignature=edaec66500db21d37602c99daa61ac983f21a6ac&filter=netmonitor&framework=1&selectedTimeRange=172800
Comment hidden (mozreview-request) |
Reporter | ||
Updated•7 years ago
|
Attachment #8923697 -
Attachment is obsolete: true
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Reporter | ||
Comment 16•7 years ago
|
||
no css transform will decrease the open/close time simple.netmonitor.open 535.19 > 510.66 -4.58% (med) simple.netmonitor.close 55.08 > 52.82 -4.1% (low) https://treeherder.mozilla.org/perf.html#/comparesubtest?originalProject=mozilla-central&newProject=try&newRevision=3730ddcda70576de724116fbd6496c99ed1e2e4b&originalSignature=edaec66500db21d37602c99daa61ac983f21a6ac&newSignature=edaec66500db21d37602c99daa61ac983f21a6ac&filter=netmonitor&framework=1&selectedTimeRange=172800
Reporter | ||
Comment 17•7 years ago
|
||
define css property in component https://treeherder.mozilla.org/#/jobs?repo=try&revision=655f1538d45f
Comment hidden (mozreview-request) |
Reporter | ||
Comment 19•7 years ago
|
||
no scroll https://treeherder.mozilla.org/#/jobs?repo=try&revision=0ecc6c65c8b9
Comment hidden (mozreview-request) |
Reporter | ||
Comment 21•7 years ago
|
||
new onboarding tests https://treeherder.mozilla.org/#/jobs?repo=try&revision=7546870c933a
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Reporter | ||
Comment 25•7 years ago
|
||
toolbar without ref.searchbox fix https://treeherder.mozilla.org/#/jobs?repo=try&revision=bf61871255b2
Comment hidden (mozreview-request) |
Reporter | ||
Comment 27•7 years ago
|
||
reselect getWaterfallScale is bad for requestsFinished perf https://treeherder.mozilla.org/perf.html#/comparesubtest?originalProject=mozilla-central&newProject=try&newRevision=f7fa27aee5d852eb537f11ac6b78d1c0db39939c&originalSignature=edaec66500db21d37602c99daa61ac983f21a6ac&newSignature=edaec66500db21d37602c99daa61ac983f21a6ac&filter=netmonitor&framework=1&selectedTimeRange=172800
Comment hidden (mozreview-request) |
Reporter | ||
Comment 29•7 years ago
|
||
comment out getStack in debugger-client http://searchfox.org/mozilla-central/source/devtools/shared/client/debugger-client.js#560 DAMP result shows simple.netmonitor.open 530.51 > 516.39 -2.66% (low) simple.netmonitor.close 56.33 > 53.07 -5.79% (low) https://treeherder.mozilla.org/perf.html#/comparesubtest?originalProject=mozilla-central&newProject=try&newRevision=8d8b67c62281ed5651293e6d1b689a41899ec47e&originalSignature=edaec66500db21d37602c99daa61ac983f21a6ac&newSignature=edaec66500db21d37602c99daa61ac983f21a6ac&framework=1&selectedTimeRange=172800 The try commit https://treeherder.mozilla.org/#/jobs?repo=try&revision=8d8b67c62281
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Reporter | ||
Updated•7 years ago
|
Attachment #8925788 -
Attachment is obsolete: true
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Reporter | ||
Comment 39•6 years ago
|
||
comment out default method, domain, contentSize column to see the damp result https://treeherder.mozilla.org/#/jobs?repo=try&revision=2835957dbbfb
Reporter | ||
Comment 40•6 years ago
|
||
removed method,domain,contentSize did increase the performance complicated.netmonitor.requestsFinished 7,486.04 > 6,607 -11.74% (high) simple.netmonitor.requestsFinished - 3.48 (low) https://treeherder.mozilla.org/perf.html#/comparesubtest?originalProject=mozilla-central&newProject=try&newRevision=2835957dbbfbb73d7750b83030949445ca2dbb88&originalSignature=edaec66500db21d37602c99daa61ac983f21a6ac&newSignature=edaec66500db21d37602c99daa61ac983f21a6ac&filter=netmonitor&framework=1&selectedTimeRange=172800
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Reporter | ||
Comment 43•6 years ago
|
||
Remove contentSize column can improve complicated.netmonitor.requestsFinished -8.24% (high, 7538 > 6917) https://treeherder.mozilla.org/perf.html#/comparesubtest?originalProject=mozilla-central&newProject=try&newRevision=5b0de1b3edc6ca739098f0d058abf80106295064&originalSignature=edaec66500db21d37602c99daa61ac983f21a6ac&newSignature=edaec66500db21d37602c99daa61ac983f21a6ac&filter=netmonitor&framework=1&selectedTimeRange=172800
Comment hidden (mozreview-request) |
Reporter | ||
Updated•6 years ago
|
Attachment #8933174 -
Attachment is obsolete: true
Reporter | ||
Updated•6 years ago
|
Attachment #8933196 -
Attachment is obsolete: true
Comment hidden (mozreview-request) |
Reporter | ||
Comment 46•6 years ago
|
||
lazyload sidepanel https://reviewboard.mozilla.org/r/194564/diff/22#index_header https://treeherder.mozilla.org/#/jobs?repo=try&revision=acfe370eb4c7 reduce -5.84% complicated.netmonitor.open and -7.47% simple.netmonitor.open time https://treeherder.mozilla.org/perf.html#/comparesubtest?originalProject=mozilla-central&newProject=try&newRevision=acfe370eb4c745558aeeed160bdbbd6279421766&originalSignature=edaec66500db21d37602c99daa61ac983f21a6ac&newSignature=edaec66500db21d37602c99daa61ac983f21a6ac&filter=netmonitor&framework=1&selectedTimeRange=172800
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Updated•6 years ago
|
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•