Split fpdfdoc/include/fpdf_doc.h into individual classes.

fpdfsdk/javascript/Icon.h

Index: fpdfsdk/javascript/Icon.h
diff --git a/fpdfsdk/javascript/Icon.h b/fpdfsdk/javascript/Icon.h
index bd125d8299b1af96fe8f05df3ad73117f3cfd59c..898cf5b3e9b409f5f24c9287d8bf13f2c99a413c 100644
--- a/fpdfsdk/javascript/Icon.h
+++ b/fpdfsdk/javascript/Icon.h
@@ -9,6 +9,8 @@
 
 #include "fpdfsdk/javascript/JS_Define.h"
 
+class CPDF_Stream;

[Lei Zhang, 2016/07/28 19:27:02]

BTW, did you know the Google C++ style guide changed at some point to say don't
bother with forward decls?
https://google.github.io/styleguide/cppguide.html#Forward_Declarations

Shall we try to start following this for new code?

It does mean more files to rebuild when changing headers, but it feels like
that's less of a concern with distributed builds and -j500.

[dsinclair, 2016/07/28 19:42:14]

Switching is probably a good idea for new code, didn't realize the style guide
said not to use them.

I'll leave the ones I have from this patch series, but will use includes in the
future.

[Lei Zhang, 2016/08/16 04:48:45]

And I missed the memo on the Chromium style guide override. :\
https://chromium.googlesource.com/chromium/src/+/master/styleguide/c++/c++.md...

+
 class Icon : public CJS_EmbedObj {
  public:
   Icon(CJS_Object* pJSObject);

[Lei Zhang, 2016/07/28 19:27:02]

explicit here and line 32.

[dsinclair, 2016/07/28 19:42:14]

Done.