852777 - Make Tilt more extensible

Status: RESOLVED FIXED

(Firefox Graveyard :: Developer Tools: 3D View, defect, P3)

Description

[Dave Townsend [:mossop]]

Attachment: WIP patch

This is a WIP and still fails a few tests but worth feedback at this point.

Basically it adds the contentWindow that Tilt is displaying to the observer notifications sent out (as well as fixing the HIDDEN notification to only get sent when necessary). It also makes the function used to set node positions a property of a presenter. This allows an extension to easily catch an early notification just after the presenter is created and supply its own node function on a per-instance basis. Much nicer than overriding the function in the JSM.

Don't bother with reviewing here, just let me know if the basic concept looks good to you. This builds on top of the patch in bug 852353.

Comment 1

[Victor Porof [:vporof][:vp]]

Comment on attachment 726979 [details] [diff] [review]
WIP patch

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

Looks good at first glance.

Comment 2

[Dave Townsend [:mossop]]

Attachment: patch rev 1

Tidied this up and added tests for the notifications and overriding the node callback function. Also been using this with an updated version of my add-on and it works nicely.

Comment 3

[Victor Porof [:vporof][:vp]]

Sorry I'll have to delay reviewing this for a few more days. I'm buried deep into network monitors at this point :)

Comment 4

[Victor Porof [:vporof][:vp]]

Comment on attachment 727938 [details] [diff] [review]
patch rev 1

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

I'm sorry this took a while. Even worse (better) is that I have nothing bad to say about this patch :) Awesome work, nice and clean. Thanks!

Comment 5

[Victor Porof [:vporof][:vp]]

https://hg.mozilla.org/integration/fx-team/rev/b4630d2ff589

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/b4630d2ff589