Andrew Comminos [:acomminos] Assignee
	Description • 9 years ago

Occasionally, the page content's scrollbar slider will disappear on GTK 3.14 using Firefox built for GTK3.

The most consistent way to reproduce this (even though it can't be reproduced 100% of the time) is to attempt to scroll a page while it loads. The slider reappears when hovering over where one would expect the scrollbar slider to be.

	Andrew Comminos [:acomminos] Assignee
	Comment 1 • 9 years ago

Strangely, if OMTC is disabled, scrollbar sliders are never visible.

	Andrew Comminos [:acomminos] Assignee
	Comment 2 • 9 years ago

The issue is caused by the fact that our tree cairo surface doesn't know that the underlying XLib surface has been modified on borrow when we draw widgets- it is believed to be clear. This patch marks it as dirty when the borrow ends, fixing the issue.

	Jeff Muizelaar [:jrmuizel]
	Comment 3 • 9 years ago

Comment on attachment 8620443 [details] [diff] [review]
Mark tree Cairo surface dirty after borrow.

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

We should also call cairo_surface_flush() in init()

	Andrew Comminos [:acomminos] Assignee
	Comment 4 • 9 years ago

Flush the surface before the borrow begins, as per cairo docs.

	Carsten Book [:Tomcat]
	Comment 5 • 9 years ago

can we get a try run here, thanks!

	Andrew Comminos [:acomminos] Assignee
	Comment 6 • 9 years ago

GTK3 try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=a2dbcc52b7d5

	Andrew Comminos [:acomminos] Assignee
	Comment 7 • 9 years ago

Changes in bug 1129873 caused GTK3 try bustage, updated try with fix:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=5fb8ef159715

	Carsten Book [:Tomcat]
	Comment 8 • 9 years ago

(In reply to Andrew Comminos [:acomminos] from comment #7)
> Changes in bug 1129873 caused GTK3 try bustage, updated try with fix:
> 
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=5fb8ef159715


this try run is full of failures like https://treeherder.mozilla.org/logviewer.html#?job_id=8451040&repo=try are those related to the patch ?

	Andrew Comminos [:acomminos] Assignee
	Comment 9 • 9 years ago

Unlikely; GTK3 builds currently aren't great on the test infrastructure (see https://treeherder.mozilla.org/#/jobs?repo=elm). Most of the failures are due to bug 1170342.

	Pulsebot
	Comment 10 • 9 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/54386de97c76

	Ryan VanderMeulen [:RyanVM]
	Comment 11 • 9 years ago

https://hg.mozilla.org/mozilla-central/rev/54386de97c76