mshtml: Implement IHTMLStyle get/put wordSpacing

Jacek Caban jacek at codeweavers.com
Fri Aug 21 06:22:57 CDT 2009


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





More information about the wine-devel mailing list