mirror of
https://github.com/facebook/react-native.git
synced 2025-11-01 09:14:26 +00:00
Paige Sun
160807d112
Summary: # Issue in iOS Before this diff, [Venice would override](https://www.internalfb.com/code/fbsource/[08e6e7a37f9ac1d33e14fc14ed763c0f8716f73a]/fbobjc/Apps/Internal/Venice/Core/RCTPerformanceLoggerUtils.mm?lines=52%2C57) the static function ReactMarker::LogTaggedMarker [created in CxxBridge](https://www.internalfb.com/code/fbsource/[08e6e7a37f9ac1d33e14fc14ed763c0f8716f73a]/xplat/js/react-native-github/React/CxxBridge/RCTCxxBridge.mm?lines=179%2C183). This means that in mixed mode they would share the Bridgeless instance of RCTPerformanceLogger [owned by Venice-only RCTInstance](https://www.internalfb.com/code/fbsource/[08e6e7a37f9ac1d33e14fc14ed763c0f8716f73a]/fbobjc/Apps/Internal/Venice/Core/RCTInstance.mm?lines=65%2C73). This is wrong because Bridge is supposed to use the instance of RCTPerformanceLogger [owned by RCTBridge](https://www.internalfb.com/code/fbsource/[73ab70b2d9e28569171b62f60e9f25744461d4d9]/xplat/js/react-native-github/React/Base/RCTBridge.m?lines=353). # Fix iOS and refactor Android 1) Add LogTaggedMarkerBridgeless to use the bridgeless RCTPerformanceLogger. 2) Use LogTaggedMarkerBridgeless to replace logTaggedMarkerWithInstanceKey. - Remove logTaggedMarkerWithInstanceKey because it always clear from the code that instanceKey is 0 for Bridge, and 1 for Bridgeless, - iOS doesn't use instanceKey and keeps separate instances of FBReactBridgeStartupLogger, FBReactWildePerfLogger, and RCTPerformanceLogger instead. This is better than using instanceKey because they are all [deallocated when Bridgeless is invalidated](https://www.internalfb.com/code/fbsource/[ea436e5ea6ae4ebc5e206197c4900022be867135]/fbobjc/Apps/Wilde/FBReactModule2/FBReactModuleAPI/FBReactModuleAPI/Exported/FBReactModule.mm?lines=1160%2C1167%2C1170). - logTaggedMarkerWithInstanceKey is only called from Venice's ReactInstance.cpp so it's easy to remove. Reviewed By: sshic Differential Revision: D32588327 fbshipit-source-id: 3151a44c9796da88fef4459b9b56946861514435
58 lines
1.5 KiB
C++
58 lines
1.5 KiB
C++
/*
|
|
* Copyright (c) Facebook, Inc. and its affiliates.
|
|
*
|
|
* This source code is licensed under the MIT license found in the
|
|
* LICENSE file in the root directory of this source tree.
|
|
*/
|
|
|
|
#pragma once
|
|
|
|
#ifdef __APPLE__
|
|
#include <functional>
|
|
#endif
|
|
|
|
namespace facebook {
|
|
namespace react {
|
|
namespace ReactMarker {
|
|
|
|
enum ReactMarkerId {
|
|
NATIVE_REQUIRE_START,
|
|
NATIVE_REQUIRE_STOP,
|
|
RUN_JS_BUNDLE_START,
|
|
RUN_JS_BUNDLE_STOP,
|
|
CREATE_REACT_CONTEXT_STOP,
|
|
JS_BUNDLE_STRING_CONVERT_START,
|
|
JS_BUNDLE_STRING_CONVERT_STOP,
|
|
NATIVE_MODULE_SETUP_START,
|
|
NATIVE_MODULE_SETUP_STOP,
|
|
REGISTER_JS_SEGMENT_START,
|
|
REGISTER_JS_SEGMENT_STOP,
|
|
REACT_INSTANCE_INIT_START,
|
|
REACT_INSTANCE_INIT_STOP
|
|
};
|
|
|
|
#ifdef __APPLE__
|
|
using LogTaggedMarker =
|
|
std::function<void(const ReactMarkerId, const char *tag)>; // Bridge only
|
|
using LogTaggedMarkerBridgeless =
|
|
std::function<void(const ReactMarkerId, const char *tag)>;
|
|
#else
|
|
typedef void (
|
|
*LogTaggedMarker)(const ReactMarkerId, const char *tag); // Bridge only
|
|
typedef void (*LogTaggedMarkerBridgeless)(const ReactMarkerId, const char *tag);
|
|
#endif
|
|
|
|
#ifndef RN_EXPORT
|
|
#define RN_EXPORT __attribute__((visibility("default")))
|
|
#endif
|
|
|
|
extern RN_EXPORT LogTaggedMarker logTaggedMarker; // Bridge only
|
|
extern RN_EXPORT LogTaggedMarker logTaggedMarkerBridgeless;
|
|
|
|
extern RN_EXPORT void logMarker(const ReactMarkerId markerId); // Bridge only
|
|
extern RN_EXPORT void logMarkerBridgeless(const ReactMarkerId markerId);
|
|
|
|
} // namespace ReactMarker
|
|
} // namespace react
|
|
} // namespace facebook
|