Move token_binding_nss.cc to net_non_nacl_sources.

Move token_binding_nss.cc to net_non_nacl_sources.

Previoulsy both token_binding_nss.cc and token_binding_openssl.cc
were compiled when compiling net for NaCl. This was causing linker
warkings about duplicate symbols.

BUG=595790
Committed: https://crrev.com/a7f891b06a0b1b278eaf241c78ce35e084b77a02
Cr-Commit-Position: refs/heads/master@{#381796}

[Sergey Ulanov, 2016-03-17 18:12:26]

sergeyu@chromium.org changed reviewers:
+ davidben@chromium.org

[davidben, 2016-03-17 18:19:44]

Why are they both getting compiled? Shouldn't this line exclude it?
https://code.google.com/p/chromium/codesearch#chromium/src/net/net_common.gyp...

[Sergey Ulanov, 2016-03-17 18:50:02]

On 2016/03/17 18:19:44, davidben wrote:
> Why are they both getting compiled? Shouldn't this line exclude it?
>
https://code.google.com/p/chromium/codesearch#chromium/src/net/net_common.gyp...

That file is not used when building net_nacl gyp target, which is in
net_nacl.gyp
Similarly with GN the openssl exclusions are applied only when is_nacl=false:
https://code.google.com/p/chromium/codesearch#chromium/src/net/BUILD.gn&l=156

[davidben, 2016-03-17 19:21:37]

On 2016/03/17 18:50:02, Sergey Ulanov wrote:
> On 2016/03/17 18:19:44, davidben wrote:
> > Why are they both getting compiled? Shouldn't this line exclude it?
> >
>
https://code.google.com/p/chromium/codesearch#chromium/src/net/net_common.gyp...
> 
> That file is not used when building net_nacl gyp target, which is in
> net_nacl.gyp
> Similarly with GN the openssl exclusions are applied only when is_nacl=false:
> https://code.google.com/p/chromium/codesearch#chromium/src/net/BUILD.gn&l=156

I see. Because rather than have is_nacl exclusions, we just had net.gypi
preapply them. lgtm

[Sergey Ulanov, 2016-03-17 19:31:40]

The CQ bit was checked by sergeyu@chromium.org

[commit-bot: I haz the power, 2016-03-17 19:32:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1807313002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1807313002/1

[commit-bot: I haz the power, 2016-03-17 21:12:39]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-17 21:13:45]

Description was changed from

==========
Move token_binding_nss.cc to net_non_nacl_sources.

Previoulsy both token_binding_nss.cc and token_binding_openssl.cc
were compiled when compiling net for NaCl. This was causing linker
warkings about duplicate symbols.

BUG=595790
==========

to

==========
Move token_binding_nss.cc to net_non_nacl_sources.

Previoulsy both token_binding_nss.cc and token_binding_openssl.cc
were compiled when compiling net for NaCl. This was causing linker
warkings about duplicate symbols.

BUG=595790

Committed: https://crrev.com/a7f891b06a0b1b278eaf241c78ce35e084b77a02
Cr-Commit-Position: refs/heads/master@{#381796}
==========

[commit-bot: I haz the power, 2016-03-17 21:13:46]

Patchset 1 (id:??) landed as
https://crrev.com/a7f891b06a0b1b278eaf241c78ce35e084b77a02
Cr-Commit-Position: refs/heads/master@{#381796}