Free mdm library after use.

Free mdm library after use.

BUG=690163
Review-Url: https://codereview.chromium.org/2722323002
Cr-Commit-Position: refs/heads/master@{#454593}
Committed: https://chromium.googlesource.com/chromium/src/+/08f4cddf58834a8d8a7d7057e0b8d020c8ef73e1

[Roger Tawa OOO till Jul 10th, 2017-03-01 21:49:34]

The CQ bit was checked by rogerta@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-01 21:50:19]

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

[Roger Tawa OOO till Jul 10th, 2017-03-01 21:53:44]

The CQ bit was checked by rogerta@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-01 21:54:20]

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

[Roger Tawa OOO till Jul 10th, 2017-03-01 21:55:28]

rogerta@chromium.org changed reviewers:
+ grt@chromium.org

[Roger Tawa OOO till Jul 10th, 2017-03-01 21:55:29]

Hi Greg,

Please take a look.  Thanks.

[commit-bot: I haz the power, 2017-03-01 23:04:28]

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

[commit-bot: I haz the power, 2017-03-01 23:04:29]

Dry run: This issue passed the CQ dry run.

[grt (UTC plus 2), 2017-03-03 09:31:35]

lgtm with a nit

https://codereview.chromium.org/2722323002/diff/20001/base/win/win_util.cc
File base/win/win_util.cc (right):

https://codereview.chromium.org/2722323002/diff/20001/base/win/win_util.cc#ne...
base/win/win_util.cc:505:
FilePath::FromUTF8Unsafe(GetNativeLibraryName("MDMRegistration")));
since this isn't cross-platform code, please simplify to:
FilePath(FILE_PATH_LITERAL("MDMRegistration.dll"))

[Roger Tawa OOO till Jul 10th, 2017-03-03 15:18:46]

Thanks Greg.

https://codereview.chromium.org/2722323002/diff/20001/base/win/win_util.cc
File base/win/win_util.cc (right):

https://codereview.chromium.org/2722323002/diff/20001/base/win/win_util.cc#ne...
base/win/win_util.cc:505:
FilePath::FromUTF8Unsafe(GetNativeLibraryName("MDMRegistration")));
On 2017/03/03 09:31:35, grt (UTC plus 1) wrote:
> since this isn't cross-platform code, please simplify to:
> FilePath(FILE_PATH_LITERAL("MDMRegistration.dll"))

Done.

[Roger Tawa OOO till Jul 10th, 2017-03-03 15:19:05]

The CQ bit was checked by rogerta@chromium.org

[Roger Tawa OOO till Jul 10th, 2017-03-03 15:19:05]

The patchset sent to the CQ was uploaded after l-g-t-m from grt@chromium.org
Link to the patchset: https://codereview.chromium.org/2722323002/#ps40001
(title: "Make windows specific")

[commit-bot: I haz the power, 2017-03-03 15:19:16]

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, 2017-03-03 16:19:53]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1488554345469570,
"parent_rev": "34094a69914e0cdc9a3215a0b53284d6f55c5b27", "commit_rev":
"08f4cddf58834a8d8a7d7057e0b8d020c8ef73e1"}

[commit-bot: I haz the power, 2017-03-03 16:20:43]

Description was changed from

==========
Free mdm library after use.

BUG=690163
==========

to

==========
Free mdm library after use.

BUG=690163

Review-Url: https://codereview.chromium.org/2722323002
Cr-Commit-Position: refs/heads/master@{#454593}
Committed:
https://chromium.googlesource.com/chromium/src/+/08f4cddf58834a8d8a7d7057e0b8...
==========

[commit-bot: I haz the power, 2017-03-03 16:20:46]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/08f4cddf58834a8d8a7d7057e0b8...