Noissue - Make the use of 'QueryInterface' calls consistent

src/plugin/PluginDomTraverserBase.h

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2015 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 163 matching lines @@
       return;
     }
 
     CComVariant vIndex(0);
     CComPtr<IDispatch> pBodyDispatch;
     if (FAILED(pBodyCollection->item(vIndex, vIndex, &pBodyDispatch)) || !pBodyDispatch)
     {
       return;
     }
 
-    if (FAILED(pBodyDispatch->QueryInterface(IID_IHTMLElement, (LPVOID*)&pBodyEl)) || !pBodyEl)
+    if (FAILED(pBodyDispatch->QueryInterface(&pBodyEl)) || !pBodyEl)
     {
       return;
     }
   }
 
   // Clear cache (if eg. refreshing) ???
   if (isMainDoc)
   {
     CComVariant vCacheIndex;
 
@@ skipping 159 matching lines @@
 
   // Get number of elements in the scope of pEl
   long allElementsCount = 0;
 
   CComPtr<IDispatch> pAllCollectionDisp;
 
   if (SUCCEEDED(pEl->get_all(&pAllCollectionDisp)) && pAllCollectionDisp)
   {
     CComPtr<IHTMLElementCollection> pAllCollection;
 
-    if (SUCCEEDED(pAllCollectionDisp->QueryInterface(IID_IHTMLElementCollection, (LPVOID*)&pAllCollection)) && pAllCollection)
+    if (SUCCEEDED(pAllCollectionDisp->QueryInterface(&pAllCollection)) && pAllCollection)
     {
       // If number of elements = cached number, return
       if (SUCCEEDED(pAllCollection->get_length(&allElementsCount)) && allElementsCount == cacheAllElementsCount)
       {
         return;
       }
     }
   }
 
   // Update cache
@@ skipping 39 matching lines @@
 
   // Iterate through children of this element
   if (allElementsCount > 0)
   {
     long childElementsCount = 0;
 
     CComPtr<IDispatch> pChildCollectionDisp;
     if (SUCCEEDED(pEl->get_children(&pChildCollectionDisp)) && pChildCollectionDisp)
     {
       CComPtr<IHTMLElementCollection> pChildCollection;
-      if (SUCCEEDED(pChildCollectionDisp->QueryInterface(IID_IHTMLElementCollection, (LPVOID*)&pChildCollection)) && pChildCollection)
+      if (SUCCEEDED(pChildCollectionDisp->QueryInterface(&pChildCollection)) && pChildCollection)
       {
         pChildCollection->get_length(&childElementsCount);
 
         CComVariant vIndex(0);
 
         // Iterate through all children
         for (long i = 0; i < childElementsCount; i++)
         {
           CComPtr<IDispatch> pChildElDispatch;
           CComVariant vRetIndex;
 
           vIndex.intVal = i;
 
           if (SUCCEEDED(pChildCollection->item(vIndex, vRetIndex, &pChildElDispatch)) && pChildElDispatch)
           {
             CComPtr<IHTMLElement> pChildEl;
-            if (SUCCEEDED(pChildElDispatch->QueryInterface(IID_IHTMLElement, (LPVOID*)&pChildEl)) && pChildEl)
+            if (SUCCEEDED(pChildElDispatch->QueryInterface(&pChildEl)) && pChildEl)
             {
               TraverseChild(pChildEl, pBrowser, indent + "  ", isCached);
             }
           }
         }
       }
     }
   }
 }
 
 
 template <class T>
 void CPluginDomTraverserBase<T>::ClearCache()
 {
   m_criticalSection.Lock();
   {
     m_cacheIndexLast = 0;
     m_cacheDocumentHasFrames.clear();
     m_cacheDocumentHasIframes.clear();
   }
   m_criticalSection.Unlock();
 }
 
 
 #endif // _PLUGIN_DOM_TRAVERSER_BASE_H_