Add support of DISPATCH_PROPERTYPUT and DISPATCH_PROPERTYPUTREF to remoting::dispatch::Invoke().

remoting/base/dispatch_win.h.pump

 $$ This is a pump file for generating file templates.  Pump is a python
 $$ script that is part of the Google Test suite of utilities.  Description
 $$ can be found here:
 $$
 $$ http://code.google.com/p/googletest/wiki/PumpManual
 
 $$ MAX_ARITY controls the number of arguments that dispatch::Invoke() supports.
 $$ It is choosen to match the number of arguments base::Bind() supports.
 $var MAX_ARITY = 7
 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
@@ skipping 152 matching lines @@
 
 
   // Invoke the method.
 
 $if ARITY > 0 [[
   DISPPARAMS disp_params = { disp_args, NULL, $(ARITY), 0 };
 ]] $else [[
   DISPPARAMS disp_params = { NULL, NULL, 0, 0 };
 ]]
 
+  // DISPATCH_PROPERTYPUT and DISPATCH_PROPERTYPUTREF require special handling.

[Wez, 2012/07/12 22:26:15]

nit: Put this in the comment above, after "Invoke the method."

[Wez, 2012/07/12 22:26:15]

Can you clarify what "special handling" means; you appear to be adding a single
named parameter to the Invoke() call?

[alexeypa (please no reviews), 2012/07/12 23:16:52]

Done.

[alexeypa (please no reviews), 2012/07/12 23:16:52]

Done.

+  DISPID dispid_named = DISPID_PROPERTYPUT;
+  if (flags == DISPATCH_PROPERTYPUT || flags == DISPATCH_PROPERTYPUTREF) {
+    disp_params.cNamedArgs = 1;
+    disp_params.rgdispidNamedArgs = &dispid_named;
+  }
+
   hr = object->Invoke(disp_id, IID_NULL, LOCALE_USER_DEFAULT, flags,
                       &disp_params, disp_result, NULL, NULL);
   if (FAILED(hr))
     return hr;
 
 $if ARITY > 0 [[
 
   // Unwrap the parameters.
 $for ARG [[
 
@@ skipping 10 matching lines @@
   return S_OK;
 }
 
 ]]
 
 } // namespace dispatch
 
 } // namespace remoting
 
 #endif // REMOTING_BASE_IDISPATCH_DRIVER_WIN_H_