[PATCH] Add multiple account saving support to KHTML/Konqueror (BUG #72317)

Filip Brcic brcha at gna.org
Sun Aug 1 15:58:47 BST 2010


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Thursday, 29. July 2010. 15.53.19 Maksim Orlovich wrote:
> Thanks for this... I'll try to take a look at this, but I feel like I
> should do my planned refactoring of the KWallet code in KHTML first,
> as it should make fixing the remaining things easier, too.

You are welcome.

I ran this patch for a week, found some errors (1: system offered to save 
password always, not only when it is new or changed + 2: you had to click on 
the password field so that it gets filled with the password because I didn't 
"setFocusNode" before setting the value) and fixed them. Also, I added support 
for writing password values using KHTMLWalletQueue as well.

What did you have in mind about refactoring KWallet in KHTML? I read (and 
changed) almost all KWallet related code in KHTML, so maybe I could be of some 
assistance.

- -- 
Filip Brcic <brcha at gna.org>
WWWeb: http://www.brcha.iz.rs
Jabber  : brcha at kdetalk.net 
GPG 0x2537C379
Fingerprint: 287D 5F24 50AA A36C 977F AC9A F1FD C7EB 2537 C379
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.14 (GNU/Linux)

iQIcBAEBAgAGBQJMVYu5AAoJEPH9x+slN8N588MP/iTVJgObjXu8snXEBxvoWXG0
EJF+8OTpY25PNE4P7BqMGV2HUbDCa7In/C9ZQWM47lF3jytPocJ0HrDmzpnNgSwK
vpJPQkS4SUFLIGJkTA5FjwDlL6aLA72qCU3ZtkBfO86TkgTWwkJjzQ+8T+ferD2k
jTOzmmWFgbAyZxB9RhZWgUwgYgidrOo5NKF0d4mjd8F9UV8mH56Gz+KKSjBQCuEC
ZCy3Rk1aeVFftOTHXcRQNv61LzG8bhTK+zS1aI8qo62mb9MHejkcv+zqXHyqdKVG
UU7mlbH9Cy5s6zoTOY0B4GiAOZ6ZqYOdwIMxXXmldCUzOGAYgqNKIapcFHlnYWZO
L6OjpgvXYKzuYlz3gmExUCE+yqHV6zjgbp5M53lv0G8FZ4X5IIjG8j/IHoMNardk
BVyUbzlm1N2BNjUVmMVcob5INOpxt1k6QcEJWIpkV9VZMuNYJ8+Pq+Vg0VSAZTtM
zMqHU1Fn6TAz1UGZ9YpQu66hKzym68fjftjZ0fhGIxpp8D1tx8okz3Dc7biB94oh
WB14A/vSLpWEc881SoBLbO7Vc5utrjWWCUxSofIVhzVx9EV9O3Y1NxbejzIybbly
oyCvz/2guU5G90i1gok20gZlFY0FHALOstRQLVC4n0Mwz13kwooXrwb6Z8Xa98wm
MnMzZwba/xmxhuyE/Ev1
=TA5i
-----END PGP SIGNATURE-----
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdelibs--khtml--enable_multiaccount_saving-2.diff
Type: text/x-patch
Size: 25749 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20100801/8a0188a6/attachment.bin>


More information about the kde-core-devel mailing list