Review fix

brave · Oct 11, 2023 · 0603bd9 · 0603bd9
1 parent 60ce4d2
commit 0603bd9
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 14 deletions.
diff --git a/browser/browser_context_keyed_service_factories.cc b/browser/browser_context_keyed_service_factories.cc
@@ -13,6 +13,7 @@
 #include "brave/browser/brave_rewards/rewards_service_factory.h"
 #include "brave/browser/brave_shields/ad_block_pref_service_factory.h"
 #include "brave/browser/brave_wallet/asset_ratio_service_factory.h"
+#include "brave/browser/brave_wallet/bitcoin_wallet_service_factory.h"
 #include "brave/browser/brave_wallet/brave_wallet_ipfs_service_factory.h"
 #include "brave/browser/brave_wallet/brave_wallet_service_factory.h"
 #include "brave/browser/brave_wallet/json_rpc_service_factory.h"
@@ -152,6 +153,9 @@ void EnsureBrowserContextKeyedServiceFactoriesBuilt() {
   brave_wallet::TxServiceFactory::GetInstance();
   brave_wallet::BraveWalletServiceFactory::GetInstance();
 
+  if (brave_wallet::IsBitcoinEnabled()) {
+    brave_wallet::BitcoinWalletServiceFactory::GetInstance();
+  }
   if (brave_wallet::IsZCashEnabled()) {
     brave_wallet::ZCashWalletServiceFactory::GetInstance();
   }

diff --git a/components/brave_wallet/browser/zcash/zcash_rpc.cc b/components/brave_wallet/browser/zcash/zcash_rpc.cc
@@ -115,24 +115,19 @@ absl::optional<std::string> ResolveSerializedMessage(
   if (grpc_response_body.size() < 5) {
     return absl::nullopt;
   }
-  if (grpc_response_body.c_str()[0] != 0) {
+  if (grpc_response_body[0] != 0) {
     // Compression is not supported yet
     return absl::nullopt;
   }
   uint32_t size = 0;
-
-  uint8_t as_bytes[4];
-  const char* size_start = &(grpc_response_body.c_str()[1]);
-  for (int i = 0; i < 4; i++) {
-    as_bytes[i] = static_cast<uint8_t>(size_start[i]);
-  }
-  base::ReadBigEndian<uint32_t>(as_bytes, &size);
+  base::ReadBigEndian(
+      reinterpret_cast<const uint8_t*>(&(grpc_response_body[1])), &size);
 
   if (grpc_response_body.size() != size + 5) {
     return absl::nullopt;
   }
 
-  return grpc_response_body.substr(5, grpc_response_body.size() - 5);
+  return grpc_response_body.substr(5);
 }
 
 }  // namespace
@@ -145,7 +140,7 @@ ZCashRpc::ZCashRpc(
 ZCashRpc::~ZCashRpc() = default;
 
 void ZCashRpc::GetUtxoList(const std::string& chain_id,
-                           const std::vector<std::string>& addresses,
+                           const std::string& address,
                            ZCashRpc::GetUtxoListCallback callback) {
   GURL request_url = MakeGetAddressUtxosURL(
       GetNetworkURL(prefs_, chain_id, mojom::CoinType::ZEC));
@@ -156,7 +151,7 @@ void ZCashRpc::GetUtxoList(const std::string& chain_id,
   }
 
   auto url_loader =
-      MakeGRPCLoader(request_url, MakeGetAddressUtxosURLParams(addresses));
+      MakeGRPCLoader(request_url, MakeGetAddressUtxosURLParams({address}));
 
   UrlLoadersList::iterator it = url_loaders_list_.insert(
       url_loaders_list_.begin(), std::move(url_loader));
@@ -174,7 +169,6 @@ void ZCashRpc::OnGetUtxosResponse(
     const std::unique_ptr<std::string> response_body) {
   auto current_loader = std::move(*it);
   url_loaders_list_.erase(it);
-  zcash::GetAddressUtxosResponse response;
   if (current_loader->NetError()) {
     std::move(callback).Run(base::unexpected("Network error"));
     return;
@@ -191,6 +185,7 @@ void ZCashRpc::OnGetUtxosResponse(
     return;
   }
 
+  zcash::GetAddressUtxosResponse response;
   if (!response.ParseFromString(message.value())) {
     std::move(callback).Run(base::unexpected("Can't parse response"));
     return;

diff --git a/components/brave_wallet/browser/zcash/zcash_rpc.h b/components/brave_wallet/browser/zcash/zcash_rpc.h
@@ -35,7 +35,7 @@ class ZCashRpc {
   ~ZCashRpc();
 
   void GetUtxoList(const std::string& chain_id,
-                   const std::vector<std::string>& addresses,
+                   const std::string& address,
                    GetUtxoListCallback callback);
 
  private:

diff --git a/...rave_wallet_ui/components/desktop/popup-modals/add-account-modal/create-account-modal.tsx b/...rave_wallet_ui/components/desktop/popup-modals/add-account-modal/create-account-modal.tsx
@@ -224,7 +224,7 @@ export const CreateAccountModal = () => {
           }
           {selectedAccountType?.coin === BraveWallet.CoinType.ZEC &&
             <SelectWrapper>
-              <Select value={bitcoinNetwork} onChange={onChangeZCashNetwork}>
+              <Select value={zcashNetwork} onChange={onChangeZCashNetwork}>
                 {ZCashNetworkTypes.map((network) => {
                   return (
                     <div data-value={network} key={network}>

diff --git a/components/brave_wallet_ui/constants/types.ts b/components/brave_wallet_ui/constants/types.ts
@@ -795,6 +795,7 @@ export interface TransactionProviderErrorRegistry {
 
 export const SupportedCoinTypes = [
   BraveWallet.CoinType.SOL,
+  BraveWallet.CoinType.ETH,
   BraveWallet.CoinType.FIL,
   BraveWallet.CoinType.BTC,
   BraveWallet.CoinType.ZEC