Re: [PATCH] mshtml: Add IHTMLAnchorElement::search propertyimplementation(try 3)

[Jacek Caban]

Hi Hao,

On 09/24/14 11:14, Hao Peng wrote:
> +#define test_anchor_search_strange(a,h,n) _test_anchor_search_strange(__LINE__,a,h,n)
> +static void _test_anchor_search_strange(unsigned line, IUnknown *unk, const char *search, const char *result)
> +{
> +    IHTMLAnchorElement *anchor = _get_anchor_iface(line, unk);
> +    BSTR str;
> +    HRESULT hres;
> +
> +    if ( ! result)
> +        result = search;
> +
> +    str = search ? a2bstr(search) : NULL;
> +    hres = IHTMLAnchorElement_put_search(anchor, str);
> +    ok_(__FILE__,line)(hres == S_OK, "put_search failed: %08x\n", hres);
> +    SysFreeString(str);
> +
> +    hres = IHTMLAnchorElement_get_search(anchor, &str);
> +    ok_(__FILE__,line)(hres == S_OK, "get_search failed: %08x\n", hres);
> +    if ( ! str)
> +        /* due to IE5's incorrect behavior, search string without leading "?" is interpreted
> +        as part of the pathname, and can not be accessed by get_search. */
> +        win_skip("incorrect behavior on IE5");

Why do you need this? Those tests are already not ran on IE5 anyway.

Jacek