kylefang/react-native
1
0
Fork 0
mirror of https://github.com/zhigang1992/react-native.git synced 2026-04-07 22:40:55 +08:00
Code Issues Packages Projects Releases Wiki Activity

Remove legacy JSC profiler

Browse Source 
Reviewed By: bnham

Differential Revision: D5433406

fbshipit-source-id: 8cbea8b9b46a0d9f29c57a5bcf605e6bb61ed8a7
This commit is contained in:
Pieter De Baets
2017-07-20 04:04:37 -07:00
committed by Facebook Github Bot
parent bf752014a9
commit ed3c018ee4
33 changed files with 0 additions and 1833 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
ReactAndroid/src/main/java/com/facebook/react/bridge/CatalystInstance.java
View File

@@ -88,10 +88,6 @@ public interface CatalystInstance
*/
void removeBridgeIdleDebugListener(NotThreadSafeBridgeIdleDebugListener listener);
boolean supportsProfiling();
void startProfiler(String title);
void stopProfiler(String title, String filename);
@VisibleForTesting
void setGlobalVariable(String propName, String jsonValue);

11
ReactAndroid/src/main/java/com/facebook/react/bridge/CatalystInstanceImpl.java
View File

@@ -429,17 +429,6 @@ public class CatalystInstanceImpl implements CatalystInstance {
@Override
public native long getJavaScriptContext();
// TODO mhorowitz: add mDestroyed checks to the next three methods
@Override
public native boolean supportsProfiling();
@Override
public native void startProfiler(String title);
@Override
public native void stopProfiler(String title, String filename);
private void incrementPendingJSCalls() {
int oldPendingCalls = mPendingJSCalls.getAndIncrement();
boolean wasIdle = oldPendingCalls == 0;

24
ReactAndroid/src/main/jni/react/jni/CatalystInstanceImpl.cpp
View File

@@ -104,9 +104,6 @@ void CatalystInstanceImpl::registerNatives() {
makeNativeMethod("setGlobalVariable", CatalystInstanceImpl::setGlobalVariable),
makeNativeMethod("getJavaScriptContext", CatalystInstanceImpl::getJavaScriptContext),
makeNativeMethod("jniHandleMemoryPressure", CatalystInstanceImpl::handleMemoryPressure),
makeNativeMethod("supportsProfiling", CatalystInstanceImpl::supportsProfiling),
makeNativeMethod("startProfiler", CatalystInstanceImpl::startProfiler),
makeNativeMethod("stopProfiler", CatalystInstanceImpl::stopProfiler),
});
JNativeRunnable::registerNatives();
@@ -266,25 +263,4 @@ void CatalystInstanceImpl::handleMemoryPressure(int pressureLevel) {
#endif
}
jboolean CatalystInstanceImpl::supportsProfiling() {
if (!instance_) {
return false;
}
return instance_->supportsProfiling();
}
void CatalystInstanceImpl::startProfiler(const std::string& title) {
if (!instance_) {
return;
}
return instance_->startProfiler(title);
}
void CatalystInstanceImpl::stopProfiler(const std::string& title, const std::string& filename) {
if (!instance_) {
return;
}
return instance_->stopProfiler(title, filename);
}
}}

3
ReactAndroid/src/main/jni/react/jni/CatalystInstanceImpl.h
View File

@@ -69,9 +69,6 @@ class CatalystInstanceImpl : public jni::HybridClass<CatalystInstanceImpl> {
std::string&& jsonValue);
jlong getJavaScriptContext();
void handleMemoryPressure(int pressureLevel);
jboolean supportsProfiling();
void startProfiler(const std::string& title);
void stopProfiler(const std::string& title, const std::string& filename);
// This should be the only long-lived strong reference, but every C++ class
// will have a weak reference.