1
0
mirror of https://github.com/kremalicious/metamask-extension.git synced 2024-12-23 09:52:26 +01:00
metamask-extension/test
Whymarrh Whitby c7fad8f400
Limit number of transactions passed outside of TransactionController (#9010)
Refs #8572
Refs #8991

This change limits the number of transactions (`txMeta`s) that are passed
outside of the `TransactionController`, resulting in shorter serialization and
deserialization times when state is moved between the background and UI
contexts.

`TransactionController#_updateMemstore`
---------------------------------------

The `currentNetworkTxList` state of the `TransactionController` is used
externally (i.e. outside of the controller) as the canonical source for
the full transaction history. Prior to this change, the method would iterate
the full transaction history and possibly return all of it.

This change limits it to `MAX_MEMSTORE_TX_LIST_SIZE` to make sure that:

1. Calls to `_updateMemstore` are fast(er)
2. Passing `currentNetworkTxList` around is fast(er)

(Shown in #8377, `_updateMemstore`, is called _frequently_ when a transaction
is pending.)

The list is iterated backwards because it is possible that new transactions are
at the end of the list. [1]

Results
-------

In profiles before this change, with ~3k transactions locally,
`PortDuplexStream._onMessage` took up to ~4.5s to complete when the set of
transactions is included. [2]

In profiles after this change, `PortDuplexStream._onMessage` took ~90ms to
complete. [3]

Before vs. after profile screenshots:

![Profile 1][2]
![Profile 2][3]

  [1]:5a3ae85b72/app/scripts/controllers/transactions/tx-state-manager.js (L172-L174)
  [2]:https://user-images.githubusercontent.com/1623628/87613203-36f51d80-c6e7-11ea-89bc-11a1cc2f3b1e.png
  [3]:https://user-images.githubusercontent.com/1623628/87613215-3bb9d180-c6e7-11ea-8d85-aff3acbd0374.png
  [8337]:https://github.com/MetaMask/metamask-extension/issues/8377
  [8572]:https://github.com/MetaMask/metamask-extension/issues/8572
  [8991]:https://github.com/MetaMask/metamask-extension/issues/8991
2020-07-16 17:52:41 -02:30
..
data Delete unused transaction history test state (#8769) 2020-06-08 17:48:28 -07:00
e2e Use eslint@6.8.0 (#8978) 2020-07-14 12:50:41 -02:30
lib Remove integration tests (#8959) 2020-07-10 12:22:36 -03:00
stub Migrate codebase to use ESM (#7730) 2020-01-09 00:04:58 -03:30
unit Limit number of transactions passed outside of TransactionController (#9010) 2020-07-16 17:52:41 -02:30
unit-global Update ESLint rules for test suite (#8023) 2020-02-11 13:21:13 -03:30
web3 Enable arrow-parens ESLint rule 2020-02-15 17:04:21 -03:30
env.js Remove cross-env (#7950) 2020-01-30 14:11:26 -03:30
helper.js Update account options menu design (#8654) 2020-05-27 12:31:53 -03:00
setup.js Remove obs-store babel register handling (#8413) 2020-04-27 07:54:42 -07:00