Fix a crash in pdf::PepperPDFHost::OnHostMsgHasUnsupportedFeature().

components/pdf/renderer/pepper_pdf_host.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_PDF_RENDERER_PEPPER_PDF_HOST_H_
 #define COMPONENTS_PDF_RENDERER_PEPPER_PDF_HOST_H_
 
 #include <stdint.h>
 
 #include <string>
+#include <vector>
 
 #include "base/compiler_specific.h"
 #include "base/macros.h"
 #include "base/strings/string16.h"
 #include "ipc/ipc_platform_file.h"
 #include "ppapi/c/ppb_image_data.h"
 #include "ppapi/c/private/ppb_pdf.h"
 #include "ppapi/host/resource_host.h"
 #include "ppapi/proxy/serialized_structs.h"
 
 struct PP_ImageDataDesc;
 struct PP_Size;
 class SkBitmap;
 
 namespace content {
 class PepperPluginInstance;
 class RendererPpapiHost;
 }
 
 namespace ppapi {
+
 class HostResource;
-}
 
-namespace ppapi {
 namespace host {
 struct HostMessageContext;
 }
-}
+}  // ppapi

[tommycli, 2016/07/22 21:47:37]

}  // namespace ppapi

Also... since there's a \n after namespace ppapi, maybe there should be a \n
before this too

[Lei Zhang, 2016/07/22 22:20:21]

Compacted.

[tommycli, 2016/07/22 22:29:11]

Hey I think it should probably still be // namespace ppapi. At least that seems
much more common in the codebase than plain // ppapi. Same with 'host'

https://cs.chromium.org/search/?q=%22%7D++//+namespace+ppapi%22&sq=package:ch...

 
 namespace pdf {
 
 class PdfAccessibilityTree;
 
 class PepperPDFHost : public ppapi::host::ResourceHost {
  public:
   class PrintClient {
    public:
     virtual ~PrintClient() {}
@@ skipping 60 matching lines @@
   std::unique_ptr<PdfAccessibilityTree> pdf_accessibility_tree_;
 
   content::RendererPpapiHost* host_;
 
   DISALLOW_COPY_AND_ASSIGN(PepperPDFHost);
 };
 
 }  // namespace pdf
 
 #endif  // COMPONENTS_PDF_RENDERER_PEPPER_PDF_HOST_H_