Re: mshtml: Implement IHTMLStyle get/put wordSpacing

[Jacek Caban]

Hi Alistair,

Alistair Leslie-Hughes wrote:
> Hi,
>
> Changelog:
>     mshtml: Implement IHTMLStyle get/put wordSpacing

static HRESULT WINAPI HTMLStyle_put_wordSpacing(IHTMLStyle *iface, VARIANT v)
 {
-    HTMLStyle *This = HTMLSTYLE_THIS(iface);
-    FIXME("(%p)->(v%d)\n", This, V_VT(&v));
-    return E_NOTIMPL;
+   HTMLStyle *This = HTMLSTYLE_THIS(iface);
+   TRACE("(%p)->(v%d)\n", This, V_VT(&v));
+   return set_nsstyle_attr_var(This->nsstyle, STYLEID_WORD_SPACING, &v, 0);

The indention here is wrong.

+    hres = IHTMLStyle_get_wordSpacing(style, &v);
+    ok(hres == S_OK, "get_wordSpacing: %08x\n", hres);
+    ok(!strcmp_wa(V_BSTR(&v), "10px"), "expected 10px = %s\n", wine_dbgstr_w(V_BSTR(&v)));
+    VariantClear(&v);

Please add a test for V_VT(&v) == VT_BSTR.

Jacek