refactor: inline StorageBackend trait methods instead of delegating

Remove unnecessary proxy method calls in StorageBackend implementations.
Directly call underlying storage operations (RocksDB/IndexedDB) instead
of delegating to Storage::get/put/delete methods.

Author: quake

light-client-lib/src/storage/db/browser.rs

@@ -1460,15 +1460,34 @@ use super::super::LightClientStorage;
 // Implementation of StorageBackend trait for IndexedDB
 impl StorageBackend for Storage {
     fn get(&self, key: Vec<u8>) -> crate::error::Result<Option<Vec<u8>>> {
-        Storage::get(self, key)
+        let values = self
+            .channel
+            .dispatch_database_command(CommandRequestWithTakeWhileAndFilterMap::Read {
+                keys: vec![key],
+            })
+            .map_err(|e| Error::Indexdb(format!("{:?}", e)))?;
+        match values {
+            DbCommandResponse::Read { values } => Ok(values.into_iter().last().unwrap()),
+            _ => unreachable!(),
+        }
     }
 
     fn put(&self, key: Vec<u8>, value: Vec<u8>) -> crate::error::Result<()> {
-        Storage::put(self, key, value)
+        self.channel
+            .dispatch_database_command(CommandRequestWithTakeWhileAndFilterMap::Put {
+                kvs: vec![KV { key, value }],
+            })
+            .map(|_| ())
+            .map_err(|e| Error::Indexdb(format!("{:?}", e)))
     }
 
     fn delete(&self, key: &[u8]) -> crate::error::Result<()> {
-        Storage::delete(self, key)
+        self.channel
+            .dispatch_database_command(CommandRequestWithTakeWhileAndFilterMap::Delete {
+                keys: vec![key.to_vec()],
+            })
+            .map(|_| ())
+            .map_err(|e| Error::Indexdb(format!("{:?}", e)))
     }
 
     fn batch(&self) -> Box<dyn BatchWriter> {

light-client-lib/src/storage/db/native.rs

@@ -1023,11 +1023,14 @@ impl StorageIterator for Storage {
 // Implementation of StorageBackend trait for RocksDB
 impl StorageBackend for Storage {
     fn get(&self, key: Vec<u8>) -> Result<Option<Vec<u8>>> {
-        Storage::get(self, key.as_slice())
+        self.db
+            .get(key.as_slice())
+            .map(|v| v.map(|vi| vi.to_vec()))
+            .map_err(Into::into)
     }
 
     fn put(&self, key: Vec<u8>, value: Vec<u8>) -> Result<()> {
-        Storage::put(self, key, value)
+        self.db.put(key, value).map_err(Into::into)
     }
 
     fn delete(&self, key: &[u8]) -> Result<()> {