Re: [PATCH v2 0/7] MR3189: Add windows.devices.geolocation.geolocator

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.devices.geolocation.geolocator/main.c:
>  return E_NOTIMPL;
>
> }
>
> +static const struct IActivationFactoryVtbl factory_vtbl;
> -
>
> static HRESULT WINAPI factory_ActivateInstance(IActivationFactory *iface, IInspectable **instance)

Also, would be nice to add a few tests for this class, specifically checking all the interfaces you implement.