Combine MSVS warning supression codes together in crypto.gyp

Combine MSVS warning supression codes
together in crypto.gyp

"msvs_disabled_warnings" code definition present in two place in the same 'target_name': 'crypto'.

Committed: https://crrev.com/22ca86792d5afe3ca862578e60009d7850cd1f94
Cr-Commit-Position: refs/heads/master@{#291903}

[ARUNKK, 2014-08-22 07:29:27]

PTAL

[vivekg, 2014-08-22 07:48:53]

On 2014/08/22 at 07:29:27, kulkarni.a wrote:
> PTAL

Please give a descriptive title e.g. "Combine MSVS warning supression codes
together in crypto.gyp"

Also keep the title as the first line of the description as only the description
entry is applied in the repository commit log.

[vivekg, 2014-08-22 07:49:15]

https://codereview.chromium.org/492963004/diff/1/crypto/crypto.gyp
File crypto/crypto.gyp (right):

https://codereview.chromium.org/492963004/diff/1/crypto/crypto.gyp#newcode98
crypto/crypto.gyp:98: ]#,
Remove the stray '#'

[ARUNKK, 2014-08-22 08:59:12]

On 2014/08/22 07:49:15, vivekg_ wrote:
> https://codereview.chromium.org/492963004/diff/1/crypto/crypto.gyp
> File crypto/crypto.gyp (right):
> 
> https://codereview.chromium.org/492963004/diff/1/crypto/crypto.gyp#newcode98
> crypto/crypto.gyp:98: ]#,
> Remove the stray '#'

Thanks for review.
Review comments are incorporated .
PTAL.

[vivekg, 2014-08-22 09:14:08]

https://codereview.chromium.org/492963004/diff/20001/crypto/crypto.gyp
File crypto/crypto.gyp (right):

https://codereview.chromium.org/492963004/diff/20001/crypto/crypto.gyp#newcode96
crypto/crypto.gyp:96: 4267,         # TODO(jschuh): crbug.com/167187 fix size_t
to int truncations.
nit: Use single whitespace for the comment "4267,         # TODO..." => "4267, #
TODO..."

see examples here:
https://code.google.com/p/chromium/codesearch#search/&q=.*%5C,%5C%20%5C%23%20...

[vivekg, 2014-08-22 09:14:31]

Also remove empty BUG= line from the description.

[ARUNKK, 2014-08-22 09:56:36]

On 2014/08/22 09:14:31, vivekg_ wrote:
> Also remove empty BUG= line from the description.

PTAL

[ARUNKK, 2014-08-26 05:53:12]

kulkarni.a@samsung.com changed reviewers:
- kphanee@chromium.org, l.gombos@samsung.com, siva.gunturi@samsung.com,
sohan.jyoti@samsung.com, vivekg@chromium.org

[ARUNKK, 2014-08-26 05:53:49]

kulkarni.a@samsung.com changed reviewers:
+ agl@chromium.org, darin@chromium.org

[ARUNKK, 2014-08-26 05:53:49]

agl@chromium.org: Please review changes in 

darin@chromium.org: Please review changes in 

PTAL

[darin (slow to review), 2014-08-26 06:40:49]

LGTM

[darin (slow to review), 2014-08-26 06:40:55]

The CQ bit was checked by darin@chromium.org

[ARUNKK, 2014-08-26 06:42:05]

The CQ bit was unchecked by kulkarni.a@samsung.com

[ARUNKK, 2014-08-26 06:42:08]

The CQ bit was checked by kulkarni.a@samsung.com

[commit-bot: I haz the power, 2014-08-26 06:42:38]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/kulkarni.a@samsung.com/492963004/40001

[ARUNKK, 2014-08-26 09:47:00]

The CQ bit was checked by kulkarni.a@samsung.com

[commit-bot: I haz the power, 2014-08-26 09:47:41]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/kulkarni.a@samsung.com/492963004/80001

[commit-bot: I haz the power, 2014-08-26 13:26:22]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  win_chromium_x64_rel_swarming on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[commit-bot: I haz the power, 2014-08-26 14:13:34]

Committed patchset #5 (80001) as 7a8f8aa1e9e72313d4b9d8f1a7091298f6be1458

[commit-bot: I haz the power, 2014-09-10 02:42:50]

Patchset 5 (id:??) landed as
https://crrev.com/22ca86792d5afe3ca862578e60009d7850cd1f94
Cr-Commit-Position: refs/heads/master@{#291903}