i#1608 client.modules: use comdlg32.dll on Windows

Commit log for first patchset:
---------------
i#1608 client.modules: use comdlg32.dll on Windows

The client.modules test was using advapi32.dll as a sample
module to load on Windows. This caused the test to fail in
environments having application compatibility enabled
and in effect during the test, because that feature causes
advapi32.dll to be loaded as a static dependency, resulting
in a missing unload event at the end of the test. This
commit changes the sample module to comdlg32.dll, which is
available on all Windows installations and will not be
loaded by application compatibility.

Fixes #1608
---------------

[bruening, 2015-02-27 14:43:31]

Please update the issue tracker to explain that it was msys.  I would also
suggest mentioning msys in the commit message.

LGTM

[Byron, 2015-02-27 18:17:41]

Committed as
https://github.com/DynamoRIO/dynamorio/commit/acc8b8c73c285db173d14a24a2e69d0...

Final commit log: 
---------------
i#1608 client.modules: use comdlg32.dll on Windows

The client.modules test was using advapi32.dll as a
sample module to load on Windows. This caused the test
to fail in environments such as msys which have
application compatibility enabled and in effect during
the test--that feature causes advapi32.dll to be loaded
as a static dependency, resulting in a missing unload
event at the end of the test. This commit changes the
sample module to comdlg32.dll, which is available on
all Windows installations and will not be loaded by the
application compatibility features

Fixes #1608

Review-URL: https://codereview.appspot.com/209880043
---------------