corrected undefined behaviour when ptr is NULL in HeapFree

corrected undefined behaviour when ptr is NULL in HeapFree.
caught one place where free(NULL) happens:
icu_56::Locale::init(const char * localeID, char canonicalize)

BUG=

Committed: https://crrev.com/dbbfbea5210e41762b1c487b1a997a2e860b7bc3
Cr-Commit-Position: refs/heads/master@{#421494}

[kruntuid, 2016-09-27 13:27:25]

Description was changed from

==========
corrected undefined behaviour when ptr is NULL in HeapFree

BUG=
==========

to

==========
corrected undefined behaviour when ptr is NULL in HeapFree.
caught one place where free(NULL) happens:
icu_56::Locale::init(const char * localeID, char canonicalize)


BUG=
==========

[kruntuid, 2016-09-27 13:27:25]

kruntuid@gmail.com changed reviewers:
+ siggi@chromium.org

[Sigurður Ásgeirsson, 2016-09-27 14:24:46]

lgtm - thanks!
"size" was a little misleading there as param name, thanks for fixing that too.

[kruntuid, 2016-09-27 14:34:26]

The CQ bit was checked by kruntuid@gmail.com

[commit-bot: I haz the power, 2016-09-27 14:34:42]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-27 14:34:43]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-27 14:34:44]

The author kruntuid@gmail.com has not signed Google Contributor License
Agreement. Please visit https://cla.developers.google.com to sign and manage
CLA.

[kruntuid, 2016-09-27 14:38:23]

The CQ bit was checked by kruntuid@gmail.com

[commit-bot: I haz the power, 2016-09-27 14:38:32]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-27 14:47:24]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-27 14:47:25]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[kruntuid, 2016-09-27 15:01:18]

kruntuid@gmail.com changed reviewers:
+ wfh@chromium.org

[kruntuid, 2016-09-27 15:01:36]

updated review

[Sigurður Ásgeirsson, 2016-09-27 15:09:17]

On 2016/09/27 15:01:36, kruntuid wrote:
> updated review

still lgtm

[Will Harris, 2016-09-27 21:20:51]

still lgtm from me too.

[kruntuid, 2016-09-28 10:25:04]

The CQ bit was checked by kruntuid@gmail.com

[commit-bot: I haz the power, 2016-09-28 10:25:25]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-28 11:05:43]

Description was changed from

==========
corrected undefined behaviour when ptr is NULL in HeapFree.
caught one place where free(NULL) happens:
icu_56::Locale::init(const char * localeID, char canonicalize)


BUG=
==========

to

==========
corrected undefined behaviour when ptr is NULL in HeapFree.
caught one place where free(NULL) happens:
icu_56::Locale::init(const char * localeID, char canonicalize)


BUG=
==========

[commit-bot: I haz the power, 2016-09-28 11:05:44]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-28 11:07:38]

Description was changed from

==========
corrected undefined behaviour when ptr is NULL in HeapFree.
caught one place where free(NULL) happens:
icu_56::Locale::init(const char * localeID, char canonicalize)


BUG=
==========

to

==========
corrected undefined behaviour when ptr is NULL in HeapFree.
caught one place where free(NULL) happens:
icu_56::Locale::init(const char * localeID, char canonicalize)

BUG=

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

[commit-bot: I haz the power, 2016-09-28 11:07:39]

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