Remove [TreatNullAs=NullString] from attributes on RTCIceCandidate

Remove [TreatNullAs=NullString] from attributes on RTCIceCandidate

Current Behaviour:
Chrome :
RTCIceCandidate.candidate = null => ""
RTCIceCandidate.sdpMid  = null => ""

Firefox :
RTCIceCandidate.candidate = null => "null"
RTCIceCandidate.sdpMid  = null => "null"

After the change:
Chrome:
RTCIceCandidate.candidate = null => "null"
RTCIceCandidate.sdpMid  = null => "null"

This aligns with the behavior on Firefox nightly

BUG=497307
Committed: https://crrev.com/11459efcc7693831a14ed164a20c8aa79fd770ba
Cr-Commit-Position: refs/heads/master@{#361087}

[tanay.c, 2015-11-04 11:56:47]

Description was changed from

==========
Remove [TreatNullAs=NullString] from attributes on RTCIceCandidate

BUG=497307
==========

to

==========
Remove [TreatNullAs=NullString] from attributes on RTCIceCandidate

Current Behaviour:
Chrome : 
RTCIceCandidate.candidate = null => ""
RTCIceCandidate.sdpMid  = null => ""

Firefox : 
RTCIceCandidate.candidate = null => "null"
RTCIceCandidate.sdpMid  = null => "null"

After the change:
Chrome:
RTCIceCandidate.candidate = null => "null"
RTCIceCandidate.sdpMid  = null => "null"

This aligns with the behavior on Firefox nightly

BUG=497307
==========

[tanay.c, 2015-11-04 11:56:47]

tanay.c@samsung.com changed reviewers:
+ philipj@opera.com

[tanay.c, 2015-11-04 11:57:33]

tanay.c@samsung.com changed reviewers:
+ mkwst@chromium.org

[tanay.c, 2015-11-04 11:57:33]

mkwst@chromium.org: Please review changes in

[tanay.c, 2015-11-04 11:57:57]

PTAL.

[philipj_slow, 2015-11-04 12:29:34]

philipj@opera.com changed reviewers:
+ peter@chromium.org, tommi@chromium.org, tommyw@chromium.org
- mkwst@chromium.org, philipj@opera.com

[philipj_slow, 2015-11-04 12:29:35]

Assigning to module owners instead. FWIW, change and tests look good to me.

[Peter Beverloo, 2015-11-06 13:02:44]

On 2015/11/04 12:29:35, philipj wrote:
> Assigning to module owners instead. FWIW, change and tests look good to me.

Agreed, but I would prefer if either Tommi or Tommy would sign off on this since
they'll be most familiar with the potential impact.

[tommi (sloooow) - chröme, 2015-11-09 21:59:53]

tommi@chromium.org changed reviewers:
+ hta@chromium.org
- tommyw@chromium.org

[tommi (sloooow) - chröme, 2015-11-09 21:59:53]

hta - wdyt?

[hta - Chromium, 2015-11-20 13:00:18]

lgtm

I see no reason not to do it. Compatibility is improved.

[Peter Beverloo, 2015-11-20 13:14:39]

rs lgtm

Thank you, hta!

[tanay.c, 2015-11-23 10:03:19]

The CQ bit was checked by tanay.c@samsung.com

[commit-bot: I haz the power, 2015-11-23 10:03:46]

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

[commit-bot: I haz the power, 2015-11-23 11:16:56]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-23 11:17:48]

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