Jared Hirsch

Results 68 comments of Jared Hirsch

@chylex have you run into this same issue when dragging regular Firefox windows, or just with the minvid window? That would likely be the difference between a wontfix (Firefox XUL...

I can take this. I did all this stuff for Screenshots.

Base commit with Talos jobs: https://treeherder.mozilla.org/#/jobs?repo=try&revision=0d75aa9246a904480470ec0eafab6fb7589d3d61

Talos run with min-vid shield study branch + moz.build file, no performance optimizations: https://treeherder.mozilla.org/#/jobs?repo=try&revision=efef136c524244832c1e962cf339cb41b8a6a068 Comparison between this and the base: https://treeherder.mozilla.org/perf.html#/compare?originalProject=try&originalRevision=0d75aa9246a904480470ec0eafab6fb7589d3d61&newProject=try&newRevision=efef136c524244832c1e962cf339cb41b8a6a068&framework=1

Here's a Talos run with the `sessionstore-windows-restored` delay added: https://treeherder.mozilla.org/#/jobs?repo=try&revision=2d4a9f2c702f7103c5a567b1673907f36e7cc3c7 Comparison between this and the base: https://treeherder.mozilla.org/perf.html#/compare?originalProject=try&originalRevision=0d75aa9246a904480470ec0eafab6fb7589d3d61&newProject=try&newRevision=2d4a9f2c702f7103c5a567b1673907f36e7cc3c7&framework=1

Oh no! I applied these patches on top of mozilla-central, but min-vid doesn't work in Nightly. I'll have to figure out which commit / branch is the correct base for...

Re-running Talos tests on top of the [FIREFOX_59_0_RELEASE tag](https://hg.mozilla.org/releases/mozilla-release/rev/FIREFOX_59_0_RELEASE). `mach` couldn't find artifacts for a commit this old, so I'm modifying the try incantation to remove the `--artifact` argument: `try:...

Hmm. Adding a startup delay does not seem to resolve the Talos perf issues: https://treeherder.mozilla.org/perf.html#/compare?originalProject=try&originalRevision=febd1a18cbce90f99f95f303c379ba0dff7abb64&newProject=try&newRevision=a649cfd06b617538dcb3db023486ff41be5c8f7c&framework=1&showOnlyImportant=1

Splitting the problem space to try to understand the source of the regressions: - Reduce URL scanning in content scripts, by only checking the page's URL, not the URLs of...

It seems like this might not get out before 60 goes to release on 5/9, so here's the same set of patches as Try runs against the [release 60 base...