Revertglog (#38)

* Revert "Updating glog version to latest(0.3.5) which is the version RN is using now (#32)"

This reverts commit 4bfaca4.

* Revert "Adding logging support on droid and logging failure message (#31)"

This reverts commit 7fd45d0.

Author: acoates-ms

ReactAndroid/src/main/java/com/facebook/react/v8executor/OnLoad.cpp

@@ -12,6 +12,12 @@
 #include <react/jni/JSLogging.h>
 #include <react/jni/ReadableNativeMap.h>
 
+namespace facebook {
+namespace v8runtime {
+      std::unique_ptr<jsi::Runtime> makeV8Runtime();
+} // namespace v8runtime
+} // namespace facebook
+
 namespace facebook {
 namespace react {
 
@@ -25,16 +31,13 @@ class V8ExecutorFactory : public JSExecutorFactory {
 
   std::unique_ptr<JSExecutor> createJSExecutor(
       std::shared_ptr<ExecutorDelegate> delegate,
-      std::shared_ptr<MessageQueueThread> jsQueue) override {    
-    
-    auto logger = std::make_shared<JSIExecutor::Logger>([](const std::string& message, unsigned int logLevel) {
-                    reactAndroidLoggingHook(message, logLevel);
-    });
-
+      std::shared_ptr<MessageQueueThread> jsQueue) override {
     return folly::make_unique<JSIExecutor>(
-      facebook::v8runtime::makeV8Runtime(m_v8Config, logger),
+      facebook::v8runtime::makeV8Runtime(m_v8Config),
       delegate,
-      *logger,
+      [](const std::string& message, unsigned int logLevel) {
+        reactAndroidLoggingHook(message, logLevel);
+      },
       JSIExecutor::defaultTimeoutInvoker,
       nullptr);
   }

ReactCommon/jsi/V8Runtime.h

@@ -60,7 +60,7 @@ namespace v8runtime {
     std::unique_ptr<const jsi::Buffer> custom_snapshot = nullptr); /*Optional*/
 
   std::unique_ptr<jsi::Runtime> makeV8Runtime();
-  std::unique_ptr<jsi::Runtime> makeV8Runtime(const folly::dynamic& v8Config, const std::shared_ptr<Logger>& logger);
+  std::unique_ptr<jsi::Runtime> makeV8Runtime(const folly::dynamic& v8Config);
 
 } // namespace v8runtime
 } // namespace facebook

ReactCommon/jsi/V8Runtime_droid.cpp

@@ -50,8 +50,7 @@ namespace facebook { namespace v8runtime {
     }
   }
 
-  V8Runtime::V8Runtime(const folly::dynamic& v8Config, const std::shared_ptr<Logger>& logger) : V8Runtime() {
-    logger_ = logger;
+  V8Runtime::V8Runtime(const folly::dynamic& v8Config) : V8Runtime() {
     isCacheEnabled_  = IsCacheEnabled(v8Config);
     shouldProduceFullCache_ = ShouldProduceFullCache(v8Config);
     shouldSetNoLazyFlag_ = ShouldSetNoLazyFlag(v8Config);

ReactCommon/jsi/V8Runtime_impl.h

@@ -43,7 +43,7 @@ namespace facebook { namespace v8runtime {
   class V8Runtime : public jsi::Runtime {
   public:
     V8Runtime();
-    V8Runtime(const folly::dynamic& v8Config, const std::shared_ptr<Logger>& logger);
+    V8Runtime(const folly::dynamic& v8Config);
 
     V8Runtime(const v8::Platform* platform, std::shared_ptr<Logger>&& logger,
       std::shared_ptr<facebook::react::MessageQueueThread>&& jsQueue, std::shared_ptr<CacheProvider>&& cacheProvider,
@@ -374,12 +374,6 @@ namespace facebook { namespace v8runtime {
     bool ExecuteString(v8::Local<v8::String> source, const jsi::Buffer* cache, v8::Local<v8::Value> name, bool report_exceptions);
     bool ExecuteString(const v8::Local<v8::String>& source, const std::string& sourceURL);
 
-    void Log(const std::string& message, const unsigned int logLevel) {
-      if (logger_) {
-        (*logger_)("V8Runtime:: " + message, logLevel);
-      }
-    }
-
     void ReportException(v8::TryCatch* try_catch);
 
     v8::Isolate* GetIsolate() const { return isolate_; }

ReactCommon/jsi/V8Runtime_shared.cpp

@@ -247,9 +247,7 @@ namespace facebook { namespace v8runtime {
     if (message.IsEmpty()) {
       // V8 didn't provide any extra information about this error; just
       // throw the exception.
-      std::string errorMessage{ "<Unknown exception>" };
-      Log(errorMessage, 3 /*LogLevel error*/);
-      throw jsi::JSError(*this, errorMessage);
+      throw jsi::JSError(*this, "<Unknown exception>");
     }
     else {
       // Print (filename):(line number): (message).
@@ -285,9 +283,7 @@ namespace facebook { namespace v8runtime {
         sstr << stack_trace_string2 << std::endl;
       }
 
-      std::string errorMessage{ sstr.str() };
-      Log(errorMessage, 3 /*LogLevel error*/);
-      throw jsi::JSError(*this, errorMessage);
+      throw jsi::JSError(*this, sstr.str());
     }
   }
 
@@ -744,7 +740,7 @@ namespace facebook { namespace v8runtime {
     return std::make_unique<V8Runtime>();
   }
 
-  std::unique_ptr<jsi::Runtime> makeV8Runtime(const folly::dynamic& v8Config, const std::shared_ptr<Logger>& logger) {
-    return std::make_unique<V8Runtime>(v8Config, logger);
+  std::unique_ptr<jsi::Runtime> makeV8Runtime(const folly::dynamic& v8Config) {
+    return std::make_unique<V8Runtime>(v8Config);
   }
-}} // namespace facebook::v8runtime
+}} // namespace facebook::v8runtime

glog/.gitignore

@@ -2,3 +2,11 @@ autom4te.cache
 glog-*.tar.gz
 packages/rpm-unknown
 packages/debian-*
+CMakeCache.txt
+CMakeFiles/
+*.cmake
+config.h
+*.sln
+*.vcxproj
+*.filters
+bazel-*

glog/AUTHORS

@@ -10,6 +10,7 @@
 
 Abhishek Dasgupta <[email protected]>
 Abhishek Parmar <[email protected]>
+Andrew Schwartzmeyer <[email protected]>
 Andy Ying <[email protected]>
 Brian Silverman <[email protected]>
 Google Inc.
@@ -20,3 +21,4 @@ romange <[email protected]>
 Sergiu Deitsch <[email protected]>
 tbennun <[email protected]>
 Teddy Reed <[email protected]>
+Zhongming Qu <[email protected]>

glog/BUILD

@@ -0,0 +1,5 @@
+licenses(['notice'])
+
+load(':bazel/glog.bzl', 'glog_library')
+
+glog_library()