Always build nss_decryptor.cc on Windows and Mac.

Always build nss_decryptor.cc on Windows and Mac.

NSS gets dlopen'd on Windows and Mac, so it can be built
in the OpenSSL port.

BUG=38883
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=282704

[davidben, 2014-07-10 23:48:54]

https://codereview.chromium.org/385873002/diff/1/chrome/chrome_utility.gypi
File chrome/chrome_utility.gypi (right):

https://codereview.chromium.org/385873002/diff/1/chrome/chrome_utility.gypi#n...
chrome/chrome_utility.gypi:131: ['OS!="android"', {
For Android, I believe this block already prevents the file from being build, so
it's not needed.

[Ryan Sleevi, 2014-07-11 01:05:34]

https://codereview.chromium.org/385873002/diff/1/chrome/chrome_utility.gypi
File chrome/chrome_utility.gypi (left):

https://codereview.chromium.org/385873002/diff/1/chrome/chrome_utility.gypi#o...
chrome/chrome_utility.gypi:124: 'utility/importer/nss_decryptor.cc',
Don't we still want to include this if on Linux, because the Linux impl doesn't
dlopen?

[davidben, 2014-07-11 13:50:50]

https://codereview.chromium.org/385873002/diff/1/chrome/chrome_utility.gypi
File chrome/chrome_utility.gypi (left):

https://codereview.chromium.org/385873002/diff/1/chrome/chrome_utility.gypi#o...
chrome/chrome_utility.gypi:124: 'utility/importer/nss_decryptor.cc',
On 2014/07/11 01:05:33, Ryan Sleevi wrote:
> Don't we still want to include this if on Linux, because the Linux impl
doesn't
> dlopen?

Hrmf, yeah, you're right. Fixed and filed crbug.com/393193 to get this working
on Linux.

[Ryan Sleevi, 2014-07-11 20:46:46]

lgtm

[davidben, 2014-07-11 20:49:13]

The CQ bit was checked by davidben@chromium.org

[commit-bot: I haz the power, 2014-07-11 20:51:30]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/davidben@chromium.org/385873002/20001

[commit-bot: I haz the power, 2014-07-11 21:37:19]

Change committed as 282704