Bug 12858 - [S60] HMNN-6YLEHJ: Browser crashes when selecting No for "Opening a secure
Summary: [S60] HMNN-6YLEHJ: Browser crashes when selecting No for "Opening a secure
Status: CLOSED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 420+
Hardware: S60 Emulator S60 3rd edition
: P1 Blocker
Assignee: Nobody
URL: http://www.op.fi
Keywords: PlatformOnly
Depends on:
Blocks:
 
Reported: 2007-02-22 13:40 PST by Yinxiu Cai
Modified: 2011-03-21 11:53 PDT (History)
2 users (show)

See Also:


Attachments
HMNN-6YLEHJ -- Browser crashes when selecting No to confirm dialog (11.95 KB, patch)
2007-02-22 15:24 PST, Yinxiu Cai
Sachin.Padma: review-
Details | Formatted Diff | Diff
Fix for bug: HMNN-6YLEHJ -- Browser crashes when selecting No to confirm dialog (13.65 KB, patch)
2007-02-23 07:51 PST, Yinxiu Cai
Sachin.Padma: review-
Details | Formatted Diff | Diff
HMNN-6YLEHJ -- Browser crashes when selecting No to confirm dialog (14.60 KB, patch)
2007-02-26 08:03 PST, Yinxiu Cai
no flags Details | Formatted Diff | Diff
HMNN-6YLEHJ -- Browser crashes when selecting No to confirm dialog (13.38 KB, patch)
2007-02-26 12:35 PST, Yinxiu Cai
no flags Details | Formatted Diff | Diff
HMNN-6YLEHJ -- Browser crashes when selecting No to confirmation dialog (13.12 KB, patch)
2007-02-27 09:29 PST, Yinxiu Cai
no flags Details | Formatted Diff | Diff
HMNN-6YLEHJ -- Browser crashes when selecting No to confirmation dialog (13.12 KB, patch)
2007-02-27 09:32 PST, Yinxiu Cai
Sachin.Padma: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yinxiu Cai 2007-02-22 13:40:21 PST
TSW ID: HMNN-6YLEHJ -- Browser crashes when selecting No for "Opening a secure connection.Content cannot be seen by anyone else."
Comment 1 Yinxiu Cai 2007-02-22 15:24:52 PST
Created attachment 13331 [details]
 HMNN-6YLEHJ -- Browser crashes when selecting No to confirm dialog

HMNN-6YLEHJ -- Browser crashes when selecting No for "Opening a secure connection.Content cannot be seen by anyone else."
Comment 2 Bradley Morrison 2007-02-22 15:32:55 PST
In terms of style, this is a nice bugg & patch. Thanks Yinxiu!!

Comment 3 Sachin Padma 2007-02-23 07:12:27 PST
Comment on attachment 13331 [details]
 HMNN-6YLEHJ -- Browser crashes when selecting No to confirm dialog

CancelTransaction needs to be done only once. Either in HandleSpecialEvent or after it returns with KErrCancel.
Comment 4 Yinxiu Cai 2007-02-23 07:51:43 PST
Created attachment 13348 [details]
Fix for bug: HMNN-6YLEHJ -- Browser crashes when selecting No to confirm dialog
Comment 5 Yinxiu Cai 2007-02-26 08:03:51 PST
Created attachment 13380 [details]
HMNN-6YLEHJ -- Browser crashes when selecting No to confirm dialog
Comment 6 Yinxiu Cai 2007-02-26 08:09:02 PST
To reproduce:
1. go to URL http://www.op.fi
2. select No for "Opening a secure connection. Content cannot be seen by anyone else."

Actual result:
Browser crashes

Expected result:
Browser should go back to the previous state.

Another way to re-produce this bug:
1. Load www. yahoo.com  web page and selct the link (with image) MAIL.
2. Select no to the confirmation query. 
  Result: Browser crashes.
Comment 7 Sachin Padma 2007-02-26 11:58:36 PST
Comment on attachment 13348 [details]
Fix for bug: HMNN-6YLEHJ -- Browser crashes when selecting No to confirm dialog 

httptortsp redirection is not taken care of
Comment 8 Yinxiu Cai 2007-02-26 12:35:43 PST
Created attachment 13384 [details]
HMNN-6YLEHJ -- Browser crashes when selecting No to confirm dialog
Comment 9 Yinxiu Cai 2007-02-27 09:29:42 PST
Created attachment 13401 [details]
HMNN-6YLEHJ -- Browser crashes when selecting No to confirmation dialog

Modified the changeLog
Comment 10 Yinxiu Cai 2007-02-27 09:32:12 PST
Created attachment 13402 [details]
HMNN-6YLEHJ -- Browser crashes when selecting No to confirmation dialog

Patch 13401 modified the changeLog
Comment 11 Sachin Padma 2007-02-27 10:59:03 PST
Comment on attachment 13402 [details]
HMNN-6YLEHJ -- Browser crashes when selecting No to confirmation dialog

r=me
Comment 12 Bradley Morrison 2007-02-28 14:28:38 PST
landed on 3.1 - r19914 
Comment 13 Bradley Morrison 2007-02-28 14:32:09 PST
and merged to s60/trunk (r19915). resolved.
Comment 14 Krishna 2007-03-04 17:24:20 PST
Verified and Closed.
Comment 15 Joel Parks 2011-03-21 11:53:13 PDT
re-purposing InTSW keyword for use by QtWebkit team