<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>100744</bug_id>
          
          <creation_ts>2012-10-30 03:59:13 -0700</creation_ts>
          <short_desc>[BlackBerry] PurgeCredential should also purge the persisted credential store</short_desc>
          <delta_ts>2012-10-30 19:02:28 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKit BlackBerry</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Charles Wei">charles.wei</reporter>
          <assigned_to name="Charles Wei">charles.wei</assigned_to>
          <cc>joenotcharles</cc>
    
    <cc>jonathan.dong.webkit</cc>
    
    <cc>mifenton</cc>
    
    <cc>rwlbuis</cc>
    
    <cc>staikos</cc>
    
    <cc>tonikitoo</cc>
    
    <cc>webkit.review.bot</cc>
    
    <cc>yong.li.webkit</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>754185</commentid>
    <comment_count>0</comment_count>
    <who name="Charles Wei">charles.wei</who>
    <bug_when>2012-10-30 03:59:13 -0700</bug_when>
    <thetext>NetworkJob::purgeCredentials() purges the credential from the memory that fails the authentication, which means that the credential is incorrect (it could be that it is incorrect from the beginning, or that it was correct but the credential was changed on server, could be through another device, etc.) and should not be used again.

We should also purge the credential record in the persistent credential storage (the CredentialBackingStore). Otherwise, the browser will read from the persistent credential storage and send the invalid credential again when challenged.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>754196</commentid>
    <comment_count>1</comment_count>
      <attachid>171415</attachid>
    <who name="Charles Wei">charles.wei</who>
    <bug_when>2012-10-30 04:06:21 -0700</bug_when>
    <thetext>Created attachment 171415
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>754198</commentid>
    <comment_count>2</comment_count>
    <who name="Charles Wei">charles.wei</who>
    <bug_when>2012-10-30 04:13:34 -0700</bug_when>
    <thetext>Internal tracking No: 233482</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>754237</commentid>
    <comment_count>3</comment_count>
      <attachid>171415</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-10-30 05:14:34 -0700</bug_when>
    <thetext>Comment on attachment 171415
Patch

Rejecting attachment 171415 from commit-queue.

Failed to run &quot;[&apos;/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch&apos;, &apos;--status-host=queues.webkit.org&apos;, &apos;-...&quot; exit_code: 2

Last 500 characters of output:
queue/Source/WebKit/chromium/third_party/skia/include --revision 6161 --non-interactive --force --accept theirs-conflict --ignore-externals&apos; in &apos;/mnt/git/webkit-commit-queue/Source/WebKit/chromium&apos;
46&gt;At revision 6161.

________ running &apos;/usr/bin/python tools/clang/scripts/update.py --mac-only&apos; in &apos;/mnt/git/webkit-commit-queue/Source/WebKit/chromium&apos;

________ running &apos;/usr/bin/python gyp_webkit&apos; in &apos;/mnt/git/webkit-commit-queue/Source/WebKit/chromium&apos;
Updating webkit projects from gyp files...

Full output: http://queues.webkit.org/results/14629506</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>754857</commentid>
    <comment_count>4</comment_count>
      <attachid>171562</attachid>
    <who name="Charles Wei">charles.wei</who>
    <bug_when>2012-10-30 18:40:18 -0700</bug_when>
    <thetext>Created attachment 171562
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>754859</commentid>
    <comment_count>5</comment_count>
      <attachid>171562</attachid>
    <who name="Charles Wei">charles.wei</who>
    <bug_when>2012-10-30 18:41:00 -0700</bug_when>
    <thetext>Comment on attachment 171562
Patch

Reviewed by George. Commit it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>754874</commentid>
    <comment_count>6</comment_count>
      <attachid>171562</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-10-30 19:02:22 -0700</bug_when>
    <thetext>Comment on attachment 171562
Patch

Clearing flags on attachment: 171562

Committed r132976: &lt;http://trac.webkit.org/changeset/132976&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>754875</commentid>
    <comment_count>7</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-10-30 19:02:28 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>171415</attachid>
            <date>2012-10-30 04:06:21 -0700</date>
            <delta_ts>2012-10-30 18:40:09 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-100744-20121030190439.patch</filename>
            <type>text/plain</type>
            <size>1997</size>
            <attacher name="Charles Wei">charles.wei</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTMyODkwCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggNDU4NWE2YjA0MDA0NjAy
YjZhNGFiNTdiZmJlOGYxYmRkMDljNzRkYS4uMDYyNTk4NzFhMWZjZWYwOGY4MmQzZTc5NmJkMzVh
MjZhYTkwYmYzMSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE5IEBACisyMDEyLTEwLTMwICBDaGFy
bGVzIFdlaSAgPGNoYXJsZXMud2VpQHRvcmNobW9iaWxlLmNvbS5jbj4KKworICAgICAgICBbQmxh
Y2tCZXJyeV0gUHVyZ2VDcmVkZW50aWFsIHNob3VsZCBhbHNvIHB1cmdlIHRoZSBwZXJzaXN0ZWQg
Y3JlZGVudGlhbCBzdG9yZQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1
Zy5jZ2k/aWQ9MTAwNzQ0CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK
KyAgICAgICAgV2hlbiBjaGFsbGVuZ2VkIGJ5IHRoZSBzZXJ2ZXIsIHdlIHdpbGwgY2hlY2sgdGhl
IGNyZWRlbnRpYWwgaW4tbWVtb3J5IGFuZCBmcm9tIHRoZSBwZXJzaXN0ZW50IHN0b3JhZ2UuCisg
ICAgICAgIFNvIHRoZSBpbnZhbGlkIGNyZWRlbnRpYWxzIHNob3VsZCBhbHNvIGJlIHB1cmdlZCBm
cm9tIHRoZSBwZXJzaXN0ZW50IGNyZWRlbnRpYWwgc3RvcmFnZTsgT3RoZXJ3aXNlLAorICAgICAg
ICBpdCB3aWxsIGJlIHNlbnQgdG8gdGhlIHNlcnZlciBuZXh0IHRpbWUgYmVlbiBjaGFsbGVuZ2Vk
LCBpbnN0ZWFkIG9mIHByb21wdGluZyB0aGUgdXNlci4KKworICAgICAgICBObyBuZXcgdGVzdHMg
KE9PUFMhKS4KKworICAgICAgICAqIHBsYXRmb3JtL25ldHdvcmsvYmxhY2tiZXJyeS9OZXR3b3Jr
Sm9iLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6Ok5ldHdvcmtKb2I6OnB1cmdlQ3JlZGVudGlhbHMp
OgorCiAyMDEyLTEwLTMwICBHYWJvciBSYXBjc2FueWkgIDxyZ2Fib3JAd2Via2l0Lm9yZz4KIAog
ICAgICAgICBPcHRpbWl6ZSB2Y2xpcCBmb3IgTkVPTiBpbiBWZWN0b3JNYXRoCmRpZmYgLS1naXQg
YS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9uZXR3b3JrL2JsYWNrYmVycnkvTmV0d29ya0pvYi5j
cHAgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9uZXR3b3JrL2JsYWNrYmVycnkvTmV0d29ya0pv
Yi5jcHAKaW5kZXggZTUyYTI0Y2JmZGVlNjVlOTY1NjJiYjlkZGZjMThlZmM5YzRlZGVkNS4uMzAx
OGFhMmNkOTM4YmFjYzcyZTJiZDM0NzdjZTJhYWQ5YzliM2ZiYSAxMDA2NDQKLS0tIGEvU291cmNl
L1dlYkNvcmUvcGxhdGZvcm0vbmV0d29yay9ibGFja2JlcnJ5L05ldHdvcmtKb2IuY3BwCisrKyBi
L1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL25ldHdvcmsvYmxhY2tiZXJyeS9OZXR3b3JrSm9iLmNw
cApAQCAtMjMsNiArMjMsNyBAQAogI2luY2x1ZGUgIkNocm9tZS5oIgogI2luY2x1ZGUgIkNocm9t
ZUNsaWVudC5oIgogI2luY2x1ZGUgIkNvb2tpZU1hbmFnZXIuaCIKKyNpbmNsdWRlICJDcmVkZW50
aWFsQmFja2luZ1N0b3JlLmgiCiAjaW5jbHVkZSAiQ3JlZGVudGlhbFN0b3JhZ2UuaCIKICNpbmNs
dWRlICJGcmFtZS5oIgogI2luY2x1ZGUgIkZyYW1lTG9hZGVyQ2xpZW50QmxhY2tCZXJyeS5oIgpA
QCAtODc0LDYgKzg3NSw5IEBAIHZvaWQgTmV0d29ya0pvYjo6cHVyZ2VDcmVkZW50aWFscygpCiAK
ICAgICBDcmVkZW50aWFsU3RvcmFnZTo6cmVtb3ZlKGNoYWxsZW5nZS5wcm90ZWN0aW9uU3BhY2Uo
KSk7CiAgICAgY2hhbGxlbmdlLnNldFN0b3JlZChmYWxzZSk7CisjaWYgRU5BQkxFKEJMQUNLQkVS
UllfQ1JFREVOVElBTF9QRVJTSVNUKQorICAgIGNyZWRlbnRpYWxCYWNraW5nU3RvcmUoKS5yZW1v
dmVMb2dpbihtX3Jlc3BvbnNlLnVybCgpLCBjaGFsbGVuZ2UucHJvdGVjdGlvblNwYWNlKCkpOwor
I2VuZGlmCiB9CiAKIGJvb2wgTmV0d29ya0pvYjo6c2hvdWxkU2VuZENsaWVudERhdGEoKSBjb25z
dAo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>171562</attachid>
            <date>2012-10-30 18:40:18 -0700</date>
            <delta_ts>2012-10-30 19:02:22 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-100744-20121031093841.patch</filename>
            <type>text/plain</type>
            <size>1964</size>
            <attacher name="Charles Wei">charles.wei</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTMyODkwCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggNDU4NWE2YjA0MDA0NjAy
YjZhNGFiNTdiZmJlOGYxYmRkMDljNzRkYS4uMzYzOGJmNzdiMTczMjMxODZkZjZkZDU0MGU3MDky
MTVhOTk0NzgxZiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDEyLTEwLTMwICBDaGFy
bGVzIFdlaSAgPGNoYXJsZXMud2VpQHRvcmNobW9iaWxlLmNvbS5jbj4KKworICAgICAgICBbQmxh
Y2tCZXJyeV0gUHVyZ2VDcmVkZW50aWFsIHNob3VsZCBhbHNvIHB1cmdlIHRoZSBwZXJzaXN0ZWQg
Y3JlZGVudGlhbCBzdG9yZQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1
Zy5jZ2k/aWQ9MTAwNzQ0CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgR2VvcmdlIFN0YWlrb3MuCisK
KyAgICAgICAgV2hlbiBjaGFsbGVuZ2VkIGJ5IHRoZSBzZXJ2ZXIsIHdlIHdpbGwgY2hlY2sgdGhl
IGNyZWRlbnRpYWwgaW4tbWVtb3J5IGFuZCBmcm9tIHRoZSBwZXJzaXN0ZW50IHN0b3JhZ2UuCisg
ICAgICAgIFNvIHRoZSBpbnZhbGlkIGNyZWRlbnRpYWxzIHNob3VsZCBhbHNvIGJlIHB1cmdlZCBm
cm9tIHRoZSBwZXJzaXN0ZW50IGNyZWRlbnRpYWwgc3RvcmFnZTsgT3RoZXJ3aXNlLAorICAgICAg
ICBpdCB3aWxsIGJlIHNlbnQgdG8gdGhlIHNlcnZlciBuZXh0IHRpbWUgYmVlbiBjaGFsbGVuZ2Vk
LCBpbnN0ZWFkIG9mIHByb21wdGluZyB0aGUgdXNlci4KKworICAgICAgICAqIHBsYXRmb3JtL25l
dHdvcmsvYmxhY2tiZXJyeS9OZXR3b3JrSm9iLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6Ok5ldHdv
cmtKb2I6OnB1cmdlQ3JlZGVudGlhbHMpOgorCiAyMDEyLTEwLTMwICBHYWJvciBSYXBjc2FueWkg
IDxyZ2Fib3JAd2Via2l0Lm9yZz4KIAogICAgICAgICBPcHRpbWl6ZSB2Y2xpcCBmb3IgTkVPTiBp
biBWZWN0b3JNYXRoCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9uZXR3b3Jr
L2JsYWNrYmVycnkvTmV0d29ya0pvYi5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9uZXR3
b3JrL2JsYWNrYmVycnkvTmV0d29ya0pvYi5jcHAKaW5kZXggZTUyYTI0Y2JmZGVlNjVlOTY1NjJi
YjlkZGZjMThlZmM5YzRlZGVkNS4uMzAxOGFhMmNkOTM4YmFjYzcyZTJiZDM0NzdjZTJhYWQ5Yzli
M2ZiYSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vbmV0d29yay9ibGFja2Jl
cnJ5L05ldHdvcmtKb2IuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL25ldHdvcmsv
YmxhY2tiZXJyeS9OZXR3b3JrSm9iLmNwcApAQCAtMjMsNiArMjMsNyBAQAogI2luY2x1ZGUgIkNo
cm9tZS5oIgogI2luY2x1ZGUgIkNocm9tZUNsaWVudC5oIgogI2luY2x1ZGUgIkNvb2tpZU1hbmFn
ZXIuaCIKKyNpbmNsdWRlICJDcmVkZW50aWFsQmFja2luZ1N0b3JlLmgiCiAjaW5jbHVkZSAiQ3Jl
ZGVudGlhbFN0b3JhZ2UuaCIKICNpbmNsdWRlICJGcmFtZS5oIgogI2luY2x1ZGUgIkZyYW1lTG9h
ZGVyQ2xpZW50QmxhY2tCZXJyeS5oIgpAQCAtODc0LDYgKzg3NSw5IEBAIHZvaWQgTmV0d29ya0pv
Yjo6cHVyZ2VDcmVkZW50aWFscygpCiAKICAgICBDcmVkZW50aWFsU3RvcmFnZTo6cmVtb3ZlKGNo
YWxsZW5nZS5wcm90ZWN0aW9uU3BhY2UoKSk7CiAgICAgY2hhbGxlbmdlLnNldFN0b3JlZChmYWxz
ZSk7CisjaWYgRU5BQkxFKEJMQUNLQkVSUllfQ1JFREVOVElBTF9QRVJTSVNUKQorICAgIGNyZWRl
bnRpYWxCYWNraW5nU3RvcmUoKS5yZW1vdmVMb2dpbihtX3Jlc3BvbnNlLnVybCgpLCBjaGFsbGVu
Z2UucHJvdGVjdGlvblNwYWNlKCkpOworI2VuZGlmCiB9CiAKIGJvb2wgTmV0d29ya0pvYjo6c2hv
dWxkU2VuZENsaWVudERhdGEoKSBjb25zdAo=
</data>

          </attachment>
      

    </bug>

</bugzilla>