[PATCH] Implement IHTMLDocument2_get_images
Alistair Leslie-Hughes
leslie_alistair at hotmail.com
Mon Nov 17 03:43:28 CST 2008
---
dlls/mshtml/htmldoc.c | 29 +++++++++++++++++++++++++++--
dlls/mshtml/tests/dom.c | 24 ++++++++++++++++++++++++
2 files changed, 51 insertions(+), 2 deletions(-)
diff --git a/dlls/mshtml/htmldoc.c b/dlls/mshtml/htmldoc.c
index 07d2fc6..0d9a0a8 100644
--- a/dlls/mshtml/htmldoc.c
+++ b/dlls/mshtml/htmldoc.c
@@ -338,8 +338,33 @@ static HRESULT WINAPI HTMLDocument_get_activeElement(IHTMLDocument2 *iface, IHTM
static HRESULT WINAPI HTMLDocument_get_images(IHTMLDocument2 *iface, IHTMLElementCollection **p)
{
HTMLDocument *This = HTMLDOC_THIS(iface);
- FIXME("(%p)->(%p)\n", This, p);
- return E_NOTIMPL;
+ nsIDOMHTMLCollection *nscoll = NULL;
+ nsresult nsres;
+
+ TRACE("(%p)->(%p)\n", This, p);
+
+ if(!p)
+ return E_INVALIDARG;
+
+ *p = NULL;
+
+ if(!This->nsdoc) {
+ WARN("NULL nsdoc\n");
+ return E_UNEXPECTED;
+ }
+
+ nsres = nsIDOMHTMLDocument_GetImages(This->nsdoc, &nscoll);
+ if(NS_FAILED(nsres)) {
+ ERR("GetImages failed: %08x\n", nsres);
+ return E_FAIL;
+ }
+
+ if(nscoll) {
+ *p = create_collection_from_htmlcol(This, (IUnknown*)HTMLDOC(This), nscoll);
+ nsIDOMElement_Release(nscoll);
+ }
+
+ return S_OK;
}
static HRESULT WINAPI HTMLDocument_get_applets(IHTMLDocument2 *iface, IHTMLElementCollection **p)
diff --git a/dlls/mshtml/tests/dom.c b/dlls/mshtml/tests/dom.c
index e9d138f..203471c 100644
--- a/dlls/mshtml/tests/dom.c
+++ b/dlls/mshtml/tests/dom.c
@@ -2743,10 +2743,24 @@ static void test_defaults(IHTMLDocument2 *doc)
IHTMLStyle *style;
long l;
HRESULT hres;
+ IHTMLElementCollection *colimages;
hres = IHTMLDocument2_get_body(doc, &elem);
ok(hres == S_OK, "get_body failed: %08x\n", hres);
+ hres = IHTMLDocument2_get_images(doc, NULL);
+ ok(hres == E_INVALIDARG, "hres %08x\n", hres);
+
+ hres = IHTMLDocument2_get_images(doc, &colimages);
+ ok(hres == S_OK, "get_images failed: %08x\n", hres);
+ if(hres == S_OK)
+ {
+ static const elem_type_t images_types[] = {ET_IMG,ET_IMG};
+
+ test_elem_collection((IUnknown*)colimages, images_types, 0);
+ IHTMLElementCollection_Release(colimages);
+ }
+
hres = IHTMLElement_QueryInterface(elem, &IID_IHTMLBodyElement, (void**)&body);
ok(hres == S_OK, "Could not get IHTMBodyElement: %08x\n", hres);
test_default_body(body);
@@ -3026,6 +3040,7 @@ static void test_elems(IHTMLDocument2 *doc)
IDispatch *disp;
long type;
HRESULT hres;
+ IHTMLElementCollection *colimages;
static const WCHAR imgidW[] = {'i','m','g','i','d',0};
static const WCHAR inW[] = {'i','n',0};
@@ -3074,6 +3089,15 @@ static void test_elems(IHTMLDocument2 *doc)
test_elem_col_item(col, xW, item_types, sizeof(item_types)/sizeof(item_types[0]));
IHTMLElementCollection_Release(col);
+ hres = IHTMLDocument2_get_images(doc, &colimages);
+ ok(hres == S_OK, "get_images failed: %08x\n", hres);
+ if(hres == S_OK)
+ {
+ static const elem_type_t images_types[] = {ET_IMG,ET_IMG};
+ test_elem_collection((IUnknown*)colimages, images_types, 1);
+ IHTMLElementCollection_Release(colimages);
+ }
+
elem = get_doc_elem(doc);
ok(hres == S_OK, "get_documentElement failed: %08x\n", hres);
hres = IHTMLElement_get_all(elem, &disp);
--
1.5.4.3
--------------080000040907030307010907--
More information about the wine-patches
mailing list