Catch stray Retains() and Releases() outside of RetainPtr<>.

The previous CLs made the code clean, so now we can mark more things
private, and add friends as appropriate.

Review-Url: https://codereview.chromium.org/2560783003

2 files changed, 8 insertions, 1 deletions

diff --git a/xfa/fxfa/parser/cxfa_widetextread.cpp b/xfa/fxfa/parser/cxfa_widetextread.cpp
index 54442df7c0..34fa6b9c5e 100644
--- a/xfa/fxfa/parser/cxfa_widetextread.cpp
+++ b/xfa/fxfa/parser/cxfa_widetextread.cpp
@@ -14,6 +14,8 @@
 CXFA_WideTextRead::CXFA_WideTextRead(const CFX_WideString& wsBuffer)
     : m_wsBuffer(wsBuffer), m_iPosition(0) {}
 
+CXFA_WideTextRead::~CXFA_WideTextRead() {}
+
 uint32_t CXFA_WideTextRead::GetAccessModes() const {
   return FX_STREAMACCESS_Read | FX_STREAMACCESS_Text;
 }
diff --git a/xfa/fxfa/parser/cxfa_widetextread.h b/xfa/fxfa/parser/cxfa_widetextread.h
index d3d3b3792a..2ccb042908 100644
--- a/xfa/fxfa/parser/cxfa_widetextread.h
+++ b/xfa/fxfa/parser/cxfa_widetextread.h
@@ -7,11 +7,13 @@
 #ifndef XFA_FXFA_PARSER_CXFA_WIDETEXTREAD_H_
 #define XFA_FXFA_PARSER_CXFA_WIDETEXTREAD_H_
 
+#include "core/fxcrt/cfx_retain_ptr.h"
 #include "xfa/fgas/crt/fgas_stream.h"
 
 class CXFA_WideTextRead : public IFGAS_Stream {
  public:
-  explicit CXFA_WideTextRead(const CFX_WideString& wsBuffer);
+  template <typename T, typename... Args>
+  friend CFX_RetainPtr<T> pdfium::MakeRetain(Args&&... args);
 
   // IFGAS_Stream
   uint32_t GetAccessModes() const override;
@@ -35,6 +37,9 @@ class CXFA_WideTextRead : public IFGAS_Stream {
   CFX_WideString GetSrcText() const;
 
  protected:
+  explicit CXFA_WideTextRead(const CFX_WideString& wsBuffer);
+  ~CXFA_WideTextRead() override;
+
   CFX_WideString m_wsBuffer;
   int32_t m_iPosition;
 };