Make the vast majority of JS headers private to src/javascript.

fpdfsdk/src/javascript/JS_Object.h

 // Copyright 2014 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
-#ifndef FPDFSDK_INCLUDE_JAVASCRIPT_JS_OBJECT_H_
-#define FPDFSDK_INCLUDE_JAVASCRIPT_JS_OBJECT_H_
+#ifndef FPDFSDK_SRC_JAVASCRIPT_JS_OBJECT_H_
+#define FPDFSDK_SRC_JAVASCRIPT_JS_OBJECT_H_
 
 #include <map>
 
 #include "../../../third_party/base/nonstd_unique_ptr.h"
-
-#include "../fsdk_define.h"       // For FX_UINT
-#include "../fsdk_mgr.h"          // For CPDFDoc_Environment
-#include "../fx_systemhandler.h"  // For IFX_SystemHandler
-#include "../jsapi/fxjs_v8.h"
+#include "../../include/fsdk_define.h"       // For FX_UINT
+#include "../../include/fsdk_mgr.h"          // For CPDFDoc_Environment

[Lei Zhang, 2015/10/05 23:39:25]

Just forward declare here.

[Tom Sepez, 2015/10/06 00:10:55]

Right, will need to move actual include to various .cpp files.

+#include "../../include/fx_systemhandler.h"  // For IFX_SystemHandler

[Lei Zhang, 2015/10/05 23:39:25]

No more IFX_SystemHandler here.

[Tom Sepez, 2015/10/06 00:10:55]

Done.

+#include "../../include/jsapi/fxjs_v8.h"
 #include "JS_Runtime.h"
 
 class CJS_Context;
 class CJS_Object;
 class CJS_Timer;
 
 class CJS_EmbedObj {
  public:
   explicit CJS_EmbedObj(CJS_Object* pJSObject);
   virtual ~CJS_EmbedObj();
@@ skipping 81 matching lines @@
   bool m_bValid;
 
   // data
   const int m_nType;  // 0:Interval; 1:TimeOut
   const FX_DWORD m_dwTimeOut;
   const CFX_WideString m_swJScript;
   CJS_Runtime* const m_pRuntime;
   CPDFDoc_Environment* const m_pApp;
 };
 
-#endif  // FPDFSDK_INCLUDE_JAVASCRIPT_JS_OBJECT_H_
+#endif  // FPDFSDK_SRC_JAVASCRIPT_JS_OBJECT_H_