refactor: remove unused methods {CDBIterator,CCoinsViewDBCursor}::GetValueSize()

These methods haven't been used since the chainstate db cache has been
switched from per-tx to per-txout model years ago (PR bitcoin#10195, commit
d342424).

Author: theStack

src/coins.h

@@ -142,7 +142,6 @@ class CCoinsViewCursor
 
     virtual bool GetKey(COutPoint &key) const = 0;
     virtual bool GetValue(Coin &coin) const = 0;
-    virtual unsigned int GetValueSize() const = 0;
 
     virtual bool Valid() const = 0;
     virtual void Next() = 0;

src/dbwrapper.h

@@ -166,11 +166,6 @@ class CDBIterator
         }
         return true;
     }
-
-    unsigned int GetValueSize() {
-        return piter->value().size();
-    }
-
 };
 
 class CDBWrapper

src/txdb.cpp

@@ -211,7 +211,6 @@ class CCoinsViewDBCursor: public CCoinsViewCursor
 
     bool GetKey(COutPoint &key) const override;
     bool GetValue(Coin &coin) const override;
-    unsigned int GetValueSize() const override;
 
     bool Valid() const override;
     void Next() override;
@@ -257,11 +256,6 @@ bool CCoinsViewDBCursor::GetValue(Coin &coin) const
     return pcursor->GetValue(coin);
 }
 
-unsigned int CCoinsViewDBCursor::GetValueSize() const
-{
-    return pcursor->GetValueSize();
-}
-
 bool CCoinsViewDBCursor::Valid() const
 {
     return keyTmp.first == DB_COIN;