Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(23)

Unified Diff: fpdfsdk/src/javascript/Document.cpp

Issue 1389783002: Make the vast majority of JS headers private to src/javascript. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: stray file. Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: fpdfsdk/src/javascript/Document.cpp
diff --git a/fpdfsdk/src/javascript/Document.cpp b/fpdfsdk/src/javascript/Document.cpp
index 796fddb5ed525eda52fc4f53ce23d3dd131aeab3..ff0c429f33eeff08ec391e0ab5ea7db4268b6929 100644
--- a/fpdfsdk/src/javascript/Document.cpp
+++ b/fpdfsdk/src/javascript/Document.cpp
@@ -4,21 +4,20 @@
// Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
-#include "../../include/javascript/JavaScript.h"
-#include "../../include/javascript/IJavaScript.h"
-#include "../../include/javascript/JS_Define.h"
-#include "../../include/javascript/JS_Object.h"
-#include "../../include/javascript/JS_Value.h"
-#include "../../include/javascript/Document.h"
-#include "../../include/javascript/JS_EventHandler.h"
-#include "../../include/javascript/JS_Context.h"
-#include "../../include/javascript/JS_Runtime.h"
-#include "../../include/javascript/app.h"
-#include "../../include/javascript/Field.h"
-#include "../../include/javascript/Icon.h"
-#include "../../include/javascript/resource.h"
+#include "Document.h"
#include "../../../third_party/base/numerics/safe_math.h"
+#include "../../include/javascript/IJavaScript.h"
+#include "JS_Define.h"
+#include "JS_Object.h"
+#include "JS_Value.h"
+#include "JS_EventHandler.h"
Lei Zhang 2015/10/05 23:39:24 nit: alphabetical order if possible. There might s
Tom Sepez 2015/10/06 00:10:54 Done, case-sensitive order (is that right?)
Lei Zhang 2015/10/06 00:13:46 I do LANG=C + unix sort command. Document.cpp in p
+#include "JS_Context.h"
+#include "JS_Runtime.h"
+#include "app.h"
+#include "Field.h"
+#include "Icon.h"
+#include "resource.h"
static v8::Isolate* GetIsolate(IFXJS_Context* cc) {
CJS_Context* pContext = (CJS_Context*)cc;

Powered by Google App Engine
This is Rietveld 408576698