Brian R. Bondy [:bbondy] Assignee
	Description • 11 years ago

It would simply the code to just call setDisplayPortForElement instead of going back to widget to get the frame metrics values from apzc.

	Brian R. Bondy [:bbondy] Assignee
	Comment 1 • 11 years ago

Should make it easier to find the source of the problem in bug 899810

	Kartikaya Gupta (email:kats@mozilla.staktrace.com)
	Comment 2 • 11 years ago

Comment on attachment 783837 [details] [diff] [review]
Simplify viewport-needs-updating code. rev1

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

LGTM. Note that you should be able to get rid of MetroCompositorParent entirely, and just move those two lines of code to wherever the compositor is created.

::: widget/windows/winrt/MetroWidget.cpp
@@ +810,2 @@
>  protected:
>    nsCOMPtr<MetroWidget> mMetroWidget;

You can also get rid mMetroWidget.

	Brian R. Bondy [:bbondy] Assignee
	Comment 4 • 11 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/df7f7c2ab2ba
https://hg.mozilla.org/integration/mozilla-inbound/rev/0eff8f365863

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

https://hg.mozilla.org/mozilla-central/rev/df7f7c2ab2ba
https://hg.mozilla.org/mozilla-central/rev/0eff8f365863