46872 - nsIWebProgressListener needs a security state change method.

Status: VERIFIED FIXED

(Core Graveyard :: Embedding: APIs, defect, P1)

Description

[Judson Valeski]

sIWebProgressListener::OnSecurityStateChange

We'll have a enum for the different security states: 
          secure 
          not secure 
          broken 
One notification happens at transition time. This one is cancellable - 
the notification method returns a PRBool. 
Another happens when we're done loading (this is really for the broken case).
This is not cancellable.

Comment 1

[Doug Turner (:dougt)]

checked in.

Comment 2

[Doug Turner (:dougt)]

checked in.  


Notes:
+ New interface is checked into
netwerk/base/public/nsISecurityEventSink.idl

+ security level flags add to the nsIWebProgressListener are not honor
yet.  RFE.

+ Deverged from inital conversation about this interface.  There will
not be any way to prevent a page transition via this interface.  I
deverged for a few reasons.  1. Current implementation does not allow
users to stop a page transition. (bad to warp an interface to fit
design... i know)  2. we post events into an event sink.  I did not want
to enforce a sync call into that event sink.  3.  Makes the interface
more complex.  This only effects page transitions (eg https -> http). 
You still will get notified, but unable to stop the progress. 
Cancelation can still be caused in the onStatusChange method.

Comment 3

[John Morrison]

verified fixed.

Comment 4

[John Morrison]

grumble.

Comment 5

[John Morrison]

fixed.

Comment 6

[John Morrison]

verified.