Make rlz_lib_test.cc and financial_ping_test.cc build on mac.

Make rlz_lib_test.cc and financial_ping_test.cc build on mac.
Committed: https://code.google.com/p/rlz/source/detail?r=101

[thakis, 2012-03-20 19:40:35]

Note that this only makes it possible to build these on mac. They aren't
compiled on mac yet because the tests don't pass yet (due to RlzValueStoreMac
being only a collection of NOTIMPLEMENTED()s).

[Roger Tawa (Google), 2012-03-20 20:12:24]

Is the plan to eventually remove these #ifdef?  why not implement the
MachineDeal class before updating these tests?

http://codereview.appspot.com/5862043/diff/1/win/lib/financial_ping_test.cc
File win/lib/financial_ping_test.cc (right):

http://codereview.appspot.com/5862043/diff/1/win/lib/financial_ping_test.cc#n...
win/lib/financial_ping_test.cc:94: , brand);
remove new line?

http://codereview.appspot.com/5862043/diff/1/win/lib/rlz_lib_test.cc
File win/lib/rlz_lib_test.cc (right):

http://codereview.appspot.com/5862043/diff/1/win/lib/rlz_lib_test.cc#newcode25
win/lib/rlz_lib_test.cc:25: #include <Windows.h>
capital W ?

[thakis, 2012-03-20 20:14:41]

On 2012/03/20 20:12:24, rogerta wrote:
> Is the plan to eventually remove these #ifdef?  why not implement the
> MachineDeal class before updating these tests?

My understanding was that MachineDeal isn't necessary on mac, since we won't
support machine deals on mac. I thought that's only used for OEM deals. Is that
not correct?

(I do plan to implement GetMachineId(), but not the rest.)

[Roger Tawa (Google), 2012-03-20 20:20:06]

lgtm

OK, makes sense about machine deals.