Comment out MemoryFootprintLinux directly to prevent build break on NDK API < 18

Summary:
    getline() only existed after NDK API 18, there is a compile error
    to support mininal API 16.
    As MemoryFootprint does not being used in JSC, comment out the
    implementation directly to prevent build break.

Author: Kudo

patches/jsc.patch

@@ -292,39 +292,24 @@ diff -aur target-org/webkit/Source/JavaScriptCore/CMakeLists.txt target/webkit/S
 +add_definitions(-DJSC_VERSION="${JSC_VERSION}")
 
 diff -aur target-org/webkit/Source/WTF/wtf/linux/MemoryFootprintLinux.cpp target/webkit/Source/WTF/wtf/linux/MemoryFootprintLinux.cpp
---- target-org/webkit/Source/WTF/wtf/linux/MemoryFootprintLinux.cpp	2018-08-24 16:08:22.000000000 +0800
-+++ target/webkit/Source/WTF/wtf/linux/MemoryFootprintLinux.cpp	2018-11-07 22:59:05.000000000 +0800
-@@ -38,6 +38,30 @@
- #if OS(LINUX)
- static const Seconds s_memoryFootprintUpdateInterval = 1_s;
+--- target-org/webkit/Source/WTF/wtf/linux/MemoryFootprintLinux.cpp     2018-11-11 23:05:48.000000000 +0800
++++ target/webkit/Source/WTF/wtf/linux/MemoryFootprintLinux.cpp 2018-11-12 23:39:22.000000000 +0800
+@@ -23,6 +23,10 @@
+  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+  */
 
-+#if defined(__ANDROID__) && __ANDROID_API__ < 18
-+//
-+// The POSIX.1 C getline isn't exported until API 18 in NDK.
-+// As the getline() used to read content from /proc/self/smaps,
-+// the content fortunately are plain text and lines are not too long.
-+// Here to simply use fgets() as fallback.
-+// Some differences further:
-+//  1. Line buffer is fixed 512 bytes.
-+//  2. Return 1 if succeed.
-+//     This is only for forEachLine() use.
-+//     Just don't want to do strlen() calculating length again.
-+//
-+static ssize_t getline(char** linep, size_t* linecapp, FILE* stream) {
-+  int fixedBufferSize = 512;
-+  if (*linep == nullptr) {
-+    *linep = reinterpret_cast<char*>(malloc(fixedBufferSize));
-+  }
-+  if (fgets(*linep, fixedBufferSize, stream) == nullptr)
-+    return -1;
++// CUSTOMIZE_REACT_NATIVE: getline() does not implemented until Android NDK API 18.
++// Since MemoryFootprint does not being used in JSC, comment out to prevent build break.
++#if !defined(CUSTOMIZE_REACT_NATIVE)
 +
-+  return 1;
-+}
-+#endif
-+
- template<typename Functor>
- static void forEachLine(FILE* file, Functor functor)
- {
+ #include "config.h"
+ #include "MemoryFootprint.h"
+
+@@ -107,3 +111,4 @@
+ }
+
+ }
++#endif // !defined(CUSTOMIZE_REACT_NATIVE)
 diff -aur target-org/webkit/Source/JavaScriptCore/Sources.txt target/webkit/Source/JavaScriptCore/Sources.txt
 --- target-org/webkit/Source/JavaScriptCore/Sources.txt 2018-11-11 23:05:40.000000000 +0800
 +++ target/webkit/Source/JavaScriptCore/Sources.txt     2018-11-12 00:03:26.000000000 +0800

scripts/compile/common.sh

@@ -141,6 +141,7 @@ COMMON_CFLAGS=" \
 -fPIC \
 -fvisibility=hidden \
 -DNDEBUG \
+-DCUSTOMIZE_REACT_NATIVE \
 $SWITCH_COMMON_CFLAGS_INTL \
 "