Bug 79650

Summary: [BlackBerry] Credential backing store implementation
Product: WebKit Reporter: Jonathan Dong <jonathan.dong.webkit>
Component: PlatformAssignee: Jonathan Dong <jonathan.dong.webkit>
Status: RESOLVED FIXED    
Severity: Normal CC: charles.wei, eric, leo.yang, rwlbuis, tonikitoo, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 80107    
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch
none
Patch
tonikitoo: review+, tonikitoo: commit-queue-
patch for cq none

Jonathan Dong
Reported 2012-02-27 02:10:14 PST
add never_remember table to store the never remember sites; change login update statement to deal with some special case like www.bing.com which would use different url when user changed credential informations; and fix some implementing bugs.
Attachments
Patch (25.61 KB, patch)
2012-02-27 02:28 PST, Jonathan Dong 		no flags
DetailsFormatted DiffDiff
Patch (212.44 KB, patch)
2012-02-27 02:37 PST, Jonathan Dong 		no flags
DetailsFormatted DiffDiff
Patch (25.63 KB, patch)
2012-02-27 02:45 PST, Jonathan Dong 		no flags
DetailsFormatted DiffDiff
Patch (26.92 KB, patch)
2012-03-01 00:07 PST, Jonathan Dong 		no flags
DetailsFormatted DiffDiff
Patch (26.91 KB, patch)
2012-03-02 00:13 PST, Jonathan Dong 		tonikitoo: review+
tonikitoo: commit-queue-
DetailsFormatted DiffDiff
patch for cq (27.24 KB, patch)
2012-03-04 17:35 PST, Jonathan Dong 		no flags
DetailsFormatted DiffDiff
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.
Jonathan Dong
Comment 1 2012-02-27 02:28:53 PST
Created attachment 128994 [details] Patch
WebKit Review Bot
Comment 2 2012-02-27 02:31:48 PST
Attachment 128994 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1 Source/WebCore/ChangeLog:10: Line contains tab character. [whitespace/tab] [5] Source/WebCore/ChangeLog:11: Line contains tab character. [whitespace/tab] [5] Source/WebCore/ChangeLog:12: Line contains tab character. [whitespace/tab] [5] Total errors found: 3 in 3 files If any of these errors are false positives, please file a bug against check-webkit-style.
Jonathan Dong
Comment 3 2012-02-27 02:37:43 PST
Created attachment 128999 [details] Patch
Jonathan Dong
Comment 4 2012-02-27 02:45:16 PST
Created attachment 129000 [details] Patch
Jonathan Dong
Comment 5 2012-03-01 00:07:19 PST
Created attachment 129649 [details] Patch
WebKit Review Bot
Comment 6 2012-03-01 00:10:47 PST
Attachment 129649 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'ChangeLog', u'LayoutTests/ChangeLog', u'La..." exit_code: 1 Tools/ChangeLog:1: ChangeLog entry has no bug number [changelog/bugnumber] [5] Total errors found: 1 in 115 files If any of these errors are false positives, please file a bug against check-webkit-style.
Jonathan Dong
Comment 7 2012-03-02 00:13:04 PST
Created attachment 129831 [details] Patch
Antonio Gomes
Comment 8 2012-03-04 10:11:56 PST
Comment on attachment 129831 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=129831&action=review > Source/WebCore/ChangeLog:13 > + Added never_remember table to store the never remember > + sites; changed login update and has statement to support > + autofill feature of multiple credential forms in the > + same page. > + > + No new tests. Patch does much more than what you are stating here. Please update before landing.
Jonathan Dong
Comment 9 2012-03-04 17:35:20 PST
Created attachment 130036 [details] patch for cq
WebKit Review Bot
Comment 10 2012-03-04 19:24:14 PST
Comment on attachment 130036 [details] patch for cq Clearing flags on attachment: 130036 Committed r109697: <http://trac.webkit.org/changeset/109697>
Eric Seidel (no email)
Comment 11 2013-01-04 00:50:54 PST
Attachment 129831 [details] was posted by a committer and has review+, assigning to Jonathan Dong for commit.
Rob Buis
Comment 12 2013-01-04 09:29:05 PST
This was landed in r109697, closing.
Note You need to log in before you can comment on or make changes to this bug.