698276 - Port Bug 686065 [Don't clear nsSessionStartup::sessionType after the session startup phase finished]

Status: ASSIGNED

(SeaMonkey :: Session Restore, defect)

Description

[Misak Khachatryan]

Attachment: patch

From parent bug:

For the new Downloads panel, having sessionType available after the session
startup phase finished can simplify some of the session restore code.

Comment 1

[[e-mail]]

Comment on attachment 570551 [details] [diff] [review]
patch

>+      if (this._sessionType != Ci.nsISessionStartup.NO_SESSION)
>+        Services.obs.removeObserver(this, "browser:purge-session-history");
>       this.init();
I don't understand this change; _sessionType defaults to Components.interfaces.nsISessionStartup.NO_SESSION and is only changed by this.init(), which is also where the observer gets added in the first place.

Comment 2

[Misak Khachatryan]

Attachment: patch v2

Comment 3

[Misak Khachatryan]

(In reply to [e-mail] from comment #1)
> Comment on attachment 570551 [details] [diff] [review] [diff] [details] [review]
> patch
> 
> I don't understand this change; _sessionType defaults to
> Components.interfaces.nsISessionStartup.NO_SESSION and is only changed by
> this.init(), which is also where the observer gets added in the first place.

It adds and removes observer if _sessionType changed from default NO_SESSION ?

Comment 4

[[e-mail]]

(In reply to Misak Khachatryan from comment #3)
> (In reply to comment #1)
> > I don't understand this change; _sessionType defaults to
> > Components.interfaces.nsISessionStartup.NO_SESSION and is only changed by
> > this.init(), which is also where the observer gets added in the first place.
> 
> It adds and removes observer if _sessionType changed from default NO_SESSION ?

But it does it before anyone changes _sessionType...

Comment 5

[Misak Khachatryan]

That(In reply to [e-mail] from comment #4)
> (In reply to Misak Khachatryan from comment #3)
> > (In reply to comment #1)
> > > I don't understand this change; _sessionType defaults to
> > > Components.interfaces.nsISessionStartup.NO_SESSION and is only changed by
> > > this.init(), which is also where the observer gets added in the first place.
> > 
> > It adds and removes observer if _sessionType changed from default NO_SESSION ?
> 
> But it does it before anyone changes _sessionType...

That's because I placed it wrong :D . It belongs to  "quit-application" case.

Comment 6

[Misak Khachatryan]

Attachment: patch v3

Comment 7

[[e-mail]]

This still doesn't look right. I'd better ask in bug 686065.

Comment 8

[[e-mail]]

Comment on attachment 571014 [details] [diff] [review]
patch v3

OK, so zpao thinks his patch might be wrong. Let's see what he comes up with.