562945 - remote=false browser binding should have similar JSON objects as remote=true

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Benjamin Stover (:stechz)]

So that Fennec doesn't have to do messy if statements, our remote=false browser should implement some things that the remote browser does for us.  In particular:
* Progress listener callbacks should have isRootWindow property on aWebProgress
* onSecurityChange should pass in security info as JSON

Comment 1

[Mark Finkle (:mfinkle) (use needinfo?)]

(In reply to comment #0)
> So that Fennec doesn't have to do messy if statements, our remote=false browser
> should implement some things that the remote browser does for us.  In
> particular:
> * Progress listener callbacks should have isRootWindow property on aWebProgress
> * onSecurityChange should pass in security info as JSON

I don't think we want to implement either of these as suggested. We don't want to change the nsIWebProgressListener callback interface. Let's do this instead:
* Get a mock .contentWindow object created and set it to aWebProgress.DOMWindow when passing the remote=true progress listener back.
* Add a .securityData property to both bindings and not pass it back in the progress listener

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

This has been fixed by various bugs