518358 - Fennec crashes when clicking "Visit Page" or "Add Exception" on Certificate NetError Page.

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Jeremias Bosch (:jbos)]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.0.14) Gecko/2009090216 Ubuntu/9.04 (jaunty) Firefox/3.0.14
Build Identifier: fennec-1.0b4pre

When calling about:blabla in fennec the ui disappears (looks like fennec crashes, but not xulrunner).

In fennec-1.0b3pre this was happening when about:neterror got called internally i.e by a invalid certificate. Within fennec-1.0b4pre this behavior when calling about:neterror is fixed, but it still happens when pressing in that error page a link or calling other valid and invalid about pages.

Reproducible: Always

Steps to Reproduce:
1. call about:blabla

Actual Results:  
Fennec Crash, Xulrunner life on. No Ui anymore.

Expected Results:  
Fennec display the pages.

Comment 1

[Jeremias Bosch (:jbos)]

Sorry forgot to mention that it is the qt-version I am talking about.

Comment 2

[Jeremias Bosch (:jbos)]

Just checked it with the newest versions of fennec and xulrunner.

The behavior regarding about:something is fixed and display a dialog.

But when you go to a page with an broken/nonconfirmable certificate you get these Add Exception page. When you press there then "Visit Page" Fennec crash / ui disappears.

Comment 3

[Jeremias Bosch (:jbos)]

I got some more informations here:

These seems to be the issue in browser.js @ 861:
       window.openDialog('chrome://pippki/content/exceptionDialog.xul',
                          '','chrome,centerscreen,modal', params);
When calling that, fennec crashes.

Comment 4

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

(In reply to comment #3)
> I got some more informations here:
> 
> These seems to be the issue in browser.js @ 861:
>        window.openDialog('chrome://pippki/content/exceptionDialog.xul',
>                           '','chrome,centerscreen,modal', params);
> When calling that, fennec crashes.


Bug 469848 (when finished) will remove that code completely.

Comment 5

[Aakash Desai [:aakashd]]

Due to comments #2-4 I'm adding a dependency to this bug for the one mentioned in comment #4 and then moving this to "new"

Comment 6

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

Fixed by bug 469848

Comment 7

[Aakash Desai [:aakashd]]

No crash on builds:

Mozilla/5.0 (X11; U; Linux armv7l; en-US; rv:1.9.2b1pre) Gecko/20091027
Fennec/1.0b5pre

and

Mozilla/5.0 (X11; U; Linux armv6l; en-US; rv:1.9.3a1pre) Gecko/20091027
Fennec/1.0b5pre