Page MenuHomePhabricator
Differential D7976

[native] Use BlobUtils for media encryption/decryption
ClosedPublic
Actions

Authored by bartek on May 25 2023, 7:28 AM.
Tags
None
Referenced Files
F4936456: D7976.id.diff
Mon, Mar 17, 9:44 PM
Unknown Object (File)
Sun, Mar 9, 9:15 AM
Unknown Object (File)
Sat, Mar 1, 11:52 PM
Unknown Object (File)
Fri, Feb 28, 3:44 PM
Unknown Object (File)
Fri, Feb 28, 3:44 PM
Unknown Object (File)
Fri, Feb 28, 3:44 PM
Unknown Object (File)
Fri, Feb 28, 3:44 PM
Unknown Object (File)
Thu, Feb 27, 3:26 PM
View All Files
Subscribers
ashoat
tomek

Details

Reviewers
atul
marcin
Commits
rCOMM0cbe6716f620: [native] Use BlobUtils for media encryption/decryption
Summary

This diff increases the performance of media encryption/decryption by using newly introduced methods instead
of basing on response.arrayBuffer() which uses slow base64 under the hood. Details are described in parent diffs and
in the task issue.

Depends on D7974

Test Plan

Ensured that media encryption/decryption still works as expected.
For these more curious, you can check performance using the following:

const startTime = Date.now();
// code from this diff here
console.log('Time:', Date.now() - startTime);

In my case, the time went down from hundreds of milliseconds to 1-2ms.

Diff Detail

Repository
rCOMM Comm
Branch
barthap/native-perf2
Lint
No Lint Coverage
Unit
No Test Coverage

Revision Contents
Changeset List

PathSize
native/
media/
9 lines

Diff 27055

View Options

native/media/encryption-utils.js

// @flow // @flow
import invariant from 'invariant'; import invariant from 'invariant';
import * as React from 'react'; import * as React from 'react';
import { useDispatch } from 'react-redux'; import { useDispatch } from 'react-redux';
import { createSelector } from 'reselect';
import { ENSCacheContext } from 'lib/components/ens-cache-provider.react'; import { ENSCacheContext } from 'lib/components/ens-cache-provider.react';
import { threadInfoSelector } from 'lib/selectors/thread-selectors';
import { createPendingSidebar } from 'lib/shared/thread-utils'; import { createPendingSidebar } from 'lib/shared/thread-utils';
import type { import type {
ComposableMessageInfo, ComposableMessageInfo,
RobotextMessageInfo, RobotextMessageInfo,
} from 'lib/types/message-types'; } from 'lib/types/message-types';
import type { ThreadInfo } from 'lib/types/thread-types'; import type { ThreadInfo, RawThreadInfo } from 'lib/types/thread-types';
import { values } from 'lib/utils/objects';
import { getDefaultTextMessageRules } from '../markdown/rules.react'; import { getDefaultTextMessageRules } from '../markdown/rules.react';
import { updateNavInfoActionType } from '../redux/action-types'; import { updateNavInfoActionType } from '../redux/action-types';
import type { AppState } from '../redux/redux-setup';
import { useSelector } from '../redux/redux-utils'; import { useSelector } from '../redux/redux-utils';
function useOnClickThread( function useOnClickThread(
thread: ?ThreadInfo, thread: ?ThreadInfo,
): (event: SyntheticEvent<HTMLElement>) => void { ): (event: SyntheticEvent<HTMLElement>) => void {
const dispatch = useDispatch(); const dispatch = useDispatch();
return React.useCallback( return React.useCallback(
(event: SyntheticEvent<HTMLElement>) => { (event: SyntheticEvent<HTMLElement>) => {
▲ Show 20 Lines • Show All 77 Lines • ▼ Show 20 Lines (event: SyntheticEvent<HTMLElement>) => {
selectedUserList: [], selectedUserList: [],
}, },
}); });
}, },
[dispatch], [dispatch],
); );
} }
function filteredCommunityThreadIDs(
communityID: string,
threadInfosObj: { +[id: string]: ThreadInfo | RawThreadInfo },
): $ReadOnlySet<string> {
const threadInfos = values(threadInfosObj);
const threadIDs = threadInfos
.filter(
thread => thread.community === communityID || thread.id === communityID,
)
.map(item => item.id);
return new Set(threadIDs);
}
const filteredCommunityThreadIDsSelector: (
state: AppState,
) => ?$ReadOnlySet<string> = createSelector(
(state: AppState) => state.communityIDFilter,
threadInfoSelector,
(
communityIDFilter: ?string,
threadInfos: { +[id: string]: ThreadInfo },
): ?$ReadOnlySet<string> => {
kamilUnsubmitted
Not Done
Inline Actions

This type could be skipped since it's obvious this will be the same as the return type of the entire selector but I saw in codebase versions with/without type in last callback so deffering to you

kamil: This type could be skipped since it's obvious this will be the same as the return type of the…
if (!communityIDFilter) {
return null;
}
return filteredCommunityThreadIDs(communityIDFilter, threadInfos);
},
);
export { export {
useOnClickThread, useOnClickThread,
useThreadIsActive, useThreadIsActive,
useOnClickPendingSidebar, useOnClickPendingSidebar,
useOnClickNewThread, useOnClickNewThread,
filteredCommunityThreadIDsSelector,
filteredCommunityThreadIDs,
}; };