[PATCH] ole32: Allow caching of icon aspects for static class caches.

[Huw Davies]

Signed-off-by: Huw Davies <huw(a)codeweavers.com>
---
 dlls/ole32/datacache.c  |  2 +-
 dlls/ole32/tests/ole2.c | 13 ++++++++++++-
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/dlls/ole32/datacache.c b/dlls/ole32/datacache.c
index c07633d960..325a98b33b 100644
--- a/dlls/ole32/datacache.c
+++ b/dlls/ole32/datacache.c
@@ -2398,7 +2398,7 @@ static HRESULT WINAPI DataCache_Cache(
         return CACHE_S_SAMECACHE;
     }
 
-    if (This->clsid_static) return DV_E_FORMATETC;
+    if (This->clsid_static && fmt_cpy.dwAspect != DVASPECT_ICON) return DV_E_FORMATETC;
 
     hr = DataCache_CreateEntry(This, &fmt_cpy, advf, FALSE, &cache_entry);
 
diff --git a/dlls/ole32/tests/ole2.c b/dlls/ole32/tests/ole2.c
index 1a4fce5b06..3bd4acce92 100644
--- a/dlls/ole32/tests/ole2.c
+++ b/dlls/ole32/tests/ole2.c
@@ -2441,13 +2441,24 @@ static void test_data_cache_cache(void)
     hr = IOleCache2_Cache( cache, &fmt, 0, &conn );
     ok( FAILED(hr), "got %08x\n", hr );
 
+    fmt.dwAspect = DVASPECT_DOCPRINT;
+    hr = IOleCache2_Cache( cache, &fmt, 0, &conn );
+    ok( FAILED(hr), "got %08x\n", hr );
+
     /* try caching another clip format */
-    fmt.dwAspect = DVASPECT_CONTENT;
     fmt.cfFormat = CF_METAFILEPICT;
+    fmt.dwAspect = DVASPECT_CONTENT;
     fmt.tymed = TYMED_MFPICT;
     hr = IOleCache2_Cache( cache, &fmt, 0, &conn );
     ok( FAILED(hr), "got %08x\n", hr );
 
+    /* As an exception, it's possible to add an icon aspect */
+    fmt.cfFormat = CF_METAFILEPICT;
+    fmt.dwAspect = DVASPECT_ICON;
+    fmt.tymed = TYMED_MFPICT;
+    hr = IOleCache2_Cache( cache, &fmt, 0, &conn );
+    ok( hr == S_OK, "got %08x\n", hr );
+
     IOleCache2_Release( cache );
 }
 
-- 
2.12.0