1144321 - Enable the vsync refresh driver on OS X

Status: RESOLVED FIXED

(Core :: Graphics, defect, P2)

Description

[Mason Chang [Inactive] [:mchang]]

+++ This bug was initially created as a clone of Bug #1144317 +++

Enable the vsync refresh driver on OS X.

Talos results:
http://compare-talos.mattn.ca/dev/?oldBranch=Try&oldRevs=c0ae486a7c8f&newBranch=Try&newRev=9f89885a1e84&submit=true

Comment 1

[Mason Chang [Inactive] [:mchang]]

Attachment: Enable vsync refresh driver on OS X

Comment 2

[Mason Chang [Inactive] [:mchang]]

Attachment: Enable vsync refresh driver on OS X

Comment 3

[Mason Chang [Inactive] [:mchang]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/f2587fd12ef1

Comment 4

[Ryan VanderMeulen [:RyanVM]]

Backed out for OSX 10.10 reftest failures.
https://hg.mozilla.org/integration/mozilla-inbound/rev/b3fcbf649765

https://treeherder.mozilla.org/logviewer.html#?job_id=7810504&repo=mozilla-inbound

Comment 5

[Mason Chang [Inactive] [:mchang]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/657c26d7d5d2

Comment 6

[Mason Chang [Inactive] [:mchang]]

Attachment: Mark box-sizing-replaced-003.xht as random on OSX 10.10

From https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=ea0035a9e043 

In case we need to.

Comment 7

[Seth Fowler [:seth] [:s2h]]

Comment on attachment 8581018 [details] [diff] [review]
Mark box-sizing-replaced-003.xht as random on OSX 10.10

Review of attachment 8581018 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good!

::: dom/base/Element.cpp
@@ +1146,5 @@
>                        const nsAString& aValue,
>                        ErrorResult& aError)
>  {
>    const nsAttrName* name = InternalGetExistingAttrNameFromQName(aName);
> +  printf_stderr("Element::SetAttribute\n");

Probably don't want this in there, right?

Comment 8

[Mason Chang [Inactive] [:mchang]]

Attachment: Mark box-sizing-replaced-003.xht as random on OSX 10.10

Thanks, updated without that printf :)

Comment 9

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/657c26d7d5d2

Comment 10

[Milan Sreckovic [:milan] (needinfo for best results)]

Release Note Request (optional, but appreciated)
[Why is this notable]: Significant changes to scrolling and async animation on OS X.  Suggest this as a combined note from this bug, as well as 1137905.
[Suggested wording]: Low level changes that lead to smoother animation and scrolling OS X.
[Links (documentation, blog post, etc)]:

Comment 11

[Mason Chang [Inactive] [:mchang]]

Note, blog post / docs incoming once it's landed and stable on Windows so we can say its live on OSX and Windows.

Comment 12

[Liz Henry (:lizzard) (relman/hg->git project)]

Added a link for the release note for 39, to https://hacks.mozilla.org/2015/01/project-silk/.