Rename "js-bundles" to "js-segments"

Differential Revision: D6244399

fbshipit-source-id: d1606d126e3b598b19fa8a0955438c8dec76f5d0
This commit is contained in:
Alex Dvornikov
2017-11-06 17:55:38 -08:00
committed by Facebook Github Bot
parent 26038f50bb
commit f1258181ee
6 changed files with 18 additions and 18 deletions

View File

@@ -210,8 +210,8 @@ public class CatalystInstanceImpl implements CatalystInstance {
jniSetSourceURL(remoteURL);
}
/* package */ void setJsBundlesDirectory(String directoryPath) {
jniSetJsBundlesDirectory(directoryPath);
/* package */ void setJsSegmentsDirectory(String directoryPath) {
jniSetJsSegmentsDirectory(directoryPath);
}
/* package */ void loadScriptFromAssets(AssetManager assetManager, String assetURL, boolean loadSynchronously) {
@@ -225,7 +225,7 @@ public class CatalystInstanceImpl implements CatalystInstance {
}
private native void jniSetSourceURL(String sourceURL);
private native void jniSetJsBundlesDirectory(String directoryPath);
private native void jniSetJsSegmentsDirectory(String directoryPath);
private native void jniLoadScriptFromAssets(AssetManager assetManager, String assetURL, boolean loadSynchronously);
private native void jniLoadScriptFromFile(String fileName, String sourceURL, boolean loadSynchronously);

View File

@@ -97,15 +97,15 @@ public abstract class JSBundleLoader {
}
/**
* This loader is used to wrap other loaders and set js bundles directory before executing
* This loader is used to wrap other loaders and set js segments directory before executing
* application script.
*/
public static JSBundleLoader createSplitBundlesLoader(
final String jsBundlesDirectory, final JSBundleLoader delegate) {
final String jsSegmentsDirectory, final JSBundleLoader delegate) {
return new JSBundleLoader() {
@Override
public String loadScript(CatalystInstanceImpl instance) {
instance.setJsBundlesDirectory(jsBundlesDirectory);
instance.setJsSegmentsDirectory(jsSegmentsDirectory);
return delegate.loadScript(instance);
}
};

View File

@@ -101,7 +101,7 @@ void CatalystInstanceImpl::registerNatives() {
makeNativeMethod("initializeBridge", CatalystInstanceImpl::initializeBridge),
makeNativeMethod("jniExtendNativeModules", CatalystInstanceImpl::extendNativeModules),
makeNativeMethod("jniSetSourceURL", CatalystInstanceImpl::jniSetSourceURL),
makeNativeMethod("jniSetJsBundlesDirectory", CatalystInstanceImpl::jniSetJsBundlesDirectory),
makeNativeMethod("jniSetJsSegmentsDirectory", CatalystInstanceImpl::jniSetJsSegmentsDirectory),
makeNativeMethod("jniLoadScriptFromAssets", CatalystInstanceImpl::jniLoadScriptFromAssets),
makeNativeMethod("jniLoadScriptFromFile", CatalystInstanceImpl::jniLoadScriptFromFile),
makeNativeMethod("jniCallJSFunction", CatalystInstanceImpl::jniCallJSFunction),
@@ -178,8 +178,8 @@ void CatalystInstanceImpl::jniSetSourceURL(const std::string& sourceURL) {
instance_->setSourceURL(sourceURL);
}
void CatalystInstanceImpl::jniSetJsBundlesDirectory(const std::string& directoryPath) {
jsBundlesDirectory_ = directoryPath;
void CatalystInstanceImpl::jniSetJsSegmentsDirectory(const std::string& directoryPath) {
jsSegmentsDirectory_ = directoryPath;
}
void CatalystInstanceImpl::jniLoadScriptFromAssets(
@@ -193,7 +193,7 @@ void CatalystInstanceImpl::jniLoadScriptFromAssets(
auto script = loadScriptFromAssets(manager, sourceURL);
if (JniJSModulesUnbundle::isUnbundle(manager, sourceURL)) {
auto bundle = JniJSModulesUnbundle::fromEntryFile(manager, sourceURL);
auto registry = jsBundlesDirectory_.empty()
auto registry = jsSegmentsDirectory_.empty()
? folly::make_unique<RAMBundleRegistry>(std::move(bundle))
: folly::make_unique<JniRAMBundleRegistry>(std::move(bundle), manager, sourceURL);
instance_->loadRAMBundle(

View File

@@ -63,7 +63,7 @@ class CatalystInstanceImpl : public jni::HybridClass<CatalystInstanceImpl> {
* Sets the path to folder where additional bundles are located.
* Needs to be invoked before "loadScript" methods are called.
*/
void jniSetJsBundlesDirectory(const std::string& directoryPath);
void jniSetJsSegmentsDirectory(const std::string& directoryPath);
void jniLoadScriptFromAssets(jni::alias_ref<JAssetManager::javaobject> assetManager, const std::string& assetURL, bool loadSynchronously);
void jniLoadScriptFromFile(const std::string& fileName, const std::string& sourceURL, bool loadSynchronously);
@@ -74,8 +74,8 @@ class CatalystInstanceImpl : public jni::HybridClass<CatalystInstanceImpl> {
jlong getJavaScriptContext();
void handleMemoryPressure(int pressureLevel);
std::string jsBundlesDirectory_;
std::string jsSegmentsDirectory_;
// This should be the only long-lived strong reference, but every C++ class
// will have a weak reference.
std::shared_ptr<Instance> instance_;