Fixed some PP_X509Certificate_Private_Field enum values.

Fixed some PP_X509Certificate_Private_Field enum values.

-Added PP_X509CERTIFICATE_PRIVATE_SUBJECT_DISTINGUISHED_NAME/PP_X509CERTIFICATE_PRIVATE_ISSUER_DISTINGUISHED_NAME which I had missed originally.
-Renamed:
PP_X509CERTIFICATE_PRIVATE_ALGORITHM_OID -> PP_X509CERTIFICATE_PRIVATE_SIGNATURE_ALGORITHM_OID
PP_X509CERTIFICATE_PRIVATE_ALGORITHM_PARAMATERS_RAW -> PP_X509CERTIFICATE_PRIVATE_SIGNATURE_ALGORITHM_PARAMATERS_RAW
To be in-line with the flash names.

This also fixes a bug in the tcp socket shared implementation which allows a resource to be released in a callback, before it is finished cleaning up.

BUG=114626
TEST=out/Debug/browser_tests --gtest_filter=*PPAPITest.*X509Certificate*


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=132013

[yzshen1, 2012-04-11 07:26:52]

https://chromiumcodereview.appspot.com/10006048/diff/8001/ppapi/api/private/p...
File ppapi/api/private/ppb_x509_certificate_private.idl (right):

https://chromiumcodereview.appspot.com/10006048/diff/8001/ppapi/api/private/p...
ppapi/api/private/ppb_x509_certificate_private.idl:41:
PP_X509CERTIFICATE_PRIVATE_ISSUER_DISTINGUISHED_NAME = 6,
If M19 and M20 have different enum values, plugins will be confused. Right?
For example, a plugin compiled with the latest pepper calls GetField() with this
...DISTINGUISHED_NAME. If it is run on M19, it actually gets the field of
....UNIQUE_ID.

[raymes, 2012-04-11 15:33:27]

Hmm, yes. But the full support will only be in M20 anyway (i.e. flash
secure sockets would break if one tried to use it with M19). Since
flash is the only user of the interface, will this be problematic to
change now without an interface rev?

On Wed, Apr 11, 2012 at 12:26 AM,  <yzshen@chromium.org> wrote:
>
>
https://chromiumcodereview.appspot.com/10006048/diff/8001/ppapi/api/private/p...
> File ppapi/api/private/ppb_x509_certificate_private.idl (right):
>
>
https://chromiumcodereview.appspot.com/10006048/diff/8001/ppapi/api/private/p...
> ppapi/api/private/ppb_x509_certificate_private.idl:41:
> PP_X509CERTIFICATE_PRIVATE_ISSUER_DISTINGUISHED_NAME = 6,
> If M19 and M20 have different enum values, plugins will be confused.
> Right?
> For example, a plugin compiled with the latest pepper calls GetField()
> with this ...DISTINGUISHED_NAME. If it is run on M19, it actually gets
> the field of ....UNIQUE_ID.
>
> https://chromiumcodereview.appspot.com/10006048/

[raymes, 2012-04-12 00:22:47]

Trung said not to modify the enums ever after it is comitted (because it could
be in Canary or something) so I changed it.

Need your LGTM. Thanks.

[yzshen1, 2012-04-12 00:25:04]

lgtm

[commit-bot: I haz the power, 2012-04-12 16:05:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/raymes@chromium.org/10006048/11040

[commit-bot: I haz the power, 2012-04-12 17:56:42]

Change committed as 132013