Remove OtScanResponseIterator, Scan handling not yet implement. Need …

…rework to not use vector

src/platform/OpenThread/GenericNetworkCommissioningThreadDriver.h

@@ -18,39 +18,11 @@
 #pragma once
 
 #include <platform/NetworkCommissioning.h>
-#include <vector>
 
 namespace chip {
 namespace DeviceLayer {
 namespace NetworkCommissioning {
 
-template <typename T>
-class OtScanResponseIterator : public Iterator<T>
-{
-public:
-    OtScanResponseIterator(std::vector<T> * apScanResponse) : mpScanResponse(apScanResponse) {}
-    size_t Count() override { return mpScanResponse != nullptr ? mpScanResponse->size() : 0; }
-    bool Next(T & item) override
-    {
-        if (mpScanResponse == nullptr || currentIterating >= mpScanResponse->size())
-        {
-            return false;
-        }
-        item = (*mpScanResponse)[currentIterating];
-        currentIterating++;
-        return true;
-    }
-    void Release() override
-    { /* nothing to do, we don't hold the ownership of the vector, and users is not expected to hold the ownership in OnFinished for
-         scan. */
-    }
-
-private:
-    size_t currentIterating = 0;
-    // Note: We cannot post a event in ScheduleLambda since std::vector is not trivial copiable.
-    std::vector<T> * mpScanResponse;
-};
-
 class GenericThreadDriver final : public ThreadDriver
 {
 public: