Closed
Bug 1207332
(skia-osx)
Opened 9 years ago
Closed 8 years ago
skia content on OS X
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
FIXED
mozilla48
Tracking | Status | |
---|---|---|
firefox47 | --- | unaffected |
firefox48 | --- | fixed |
People
(Reporter: jrmuizel, Assigned: mchang)
References
(Depends on 1 open bug)
Details
(Whiteboard: [gfx-noted])
Attachments
(3 files)
1.12 KB,
patch
|
lsalzman
:
review+
|
Details | Diff | Splinter Review |
3.54 KB,
patch
|
lsalzman
:
review+
|
Details | Diff | Splinter Review |
9.12 KB,
patch
|
lsalzman
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Reporter | ||
Updated•9 years ago
|
Depends on: skia-reftest
Reporter | ||
Updated•9 years ago
|
Blocks: Backendnuken
Reporter | ||
Updated•9 years ago
|
Alias: skia-osx
Updated•9 years ago
|
Whiteboard: [gfx-noted]
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → mchang
Assignee | ||
Comment 1•8 years ago
|
||
Talos perfherder results with CG vs Skia content. Some pretty nice wins! https://treeherder.mozilla.org/perf.html#/compare?originalProject=try&originalRevision=451ff0ecf4b1&newProject=try&newRevision=c291279314ef&framework=1
Assignee | ||
Comment 2•8 years ago
|
||
Enable skia content on OS X on nightly. Will ride the trains starting in Gecko 48, which is early March.
Attachment #8720510 -
Flags: review?(lsalzman)
Assignee | ||
Comment 3•8 years ago
|
||
Some reftest fuzzing for skia content since updating to master.
Attachment #8720511 -
Flags: review?(lsalzman)
Assignee | ||
Comment 4•8 years ago
|
||
Try result for part 2 fuzzing: https://treeherder.mozilla.org/#/jobs?repo=try&revision=e9729b52db24
Updated•8 years ago
|
Attachment #8720511 -
Flags: review?(lsalzman) → review+
Comment 5•8 years ago
|
||
Comment on attachment 8720510 [details] [diff] [review] Enable Skia content on OS X on Nightly only We're basically ready to pref this on, but we decided to wait till 48 before doing so to give adequate time to test.
Attachment #8720510 -
Flags: review?(lsalzman) → review+
Assignee | ||
Updated•8 years ago
|
Keywords: leave-open
Comment 7•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ac6007041aa7
Assignee | ||
Comment 8•8 years ago
|
||
Talos results prior to pushing - https://treeherder.mozilla.org/perf.html#/compare?originalProject=try&originalRevision=986ab0f759cb&newProject=try&newRevision=de3e6e89ecfd&framework=1&showOnlyConfident=1 Looks like bug 1239151 is still valid.
Assignee | ||
Comment 9•8 years ago
|
||
Attachment #8729736 -
Flags: review?(lsalzman)
Updated•8 years ago
|
Attachment #8729736 -
Flags: review?(lsalzman) → review+
Assignee | ||
Comment 10•8 years ago
|
||
Successful try for part 3: https://treeherder.mozilla.org/#/jobs?repo=try&revision=ac1e0bd7a66f
Assignee | ||
Comment 13•8 years ago
|
||
OS X try - https://treeherder.mozilla.org/#/jobs?repo=try&revision=03cde47968cd - Reftests were fixed in part 3.
Comment 15•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/041d819ece2e https://hg.mozilla.org/mozilla-central/rev/66ee9fcbbbd3 https://hg.mozilla.org/mozilla-central/rev/9766c66da771
Comment 17•8 years ago
|
||
big wins here: https://treeherder.mozilla.org/perf.html#/alerts?id=534 and a small regression in the Main_RSS- not sure if that is expected, but I did see some bugs to reduce the memory/cache. thanks for the big win.
Comment 18•8 years ago
|
||
main rss regression is now on aurora.
Assignee | ||
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•8 years ago
|
Updated•8 years ago
|
Keywords: leave-open
OS: Unspecified → Mac OS X
Hardware: Unspecified → All
Version: unspecified → Trunk
Updated•8 years ago
|
Summary: skia content on OS X → skia content on OS X on nightly
Assignee | ||
Comment 19•8 years ago
|
||
This is riding the trains, I just was bad and landed the patch without #ifdef NIGHTLY_ONLY versus what's here in the patch.
Summary: skia content on OS X on nightly → skia content on OS X
You need to log in
before you can comment on or make changes to this bug.
Description
•