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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: xfa/fgas/font/cfgas_fontmgr.h

Issue 2562563002: Properly ref count IFX_FileAccess. (Closed)
Patch Set: Created 4 years 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
« core/fxcrt/fx_extension.cpp ('K') | « core/fxcrt/fx_stream.h ('k') | xfa/fgas/font/cfgas_fontmgr.cpp » ('j') | xfa/fgas/font/cfgas_fontmgr.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: xfa/fgas/font/cfgas_fontmgr.h
diff --git a/xfa/fgas/font/cfgas_fontmgr.h b/xfa/fgas/font/cfgas_fontmgr.h
index 28a8bb528a34e7e718d8b0f112222980d8e36a42..d3eb8f1a857b48914c3a2331040e2bc07a18d1b0 100644
--- a/xfa/fgas/font/cfgas_fontmgr.h
+++ b/xfa/fgas/font/cfgas_fontmgr.h
@@ -179,7 +179,7 @@ class CFX_FontSourceEnum_File {
~CFX_FontSourceEnum_File();
FX_POSITION GetStartPosition();
- IFX_FileAccess* GetNext(FX_POSITION& pos);
+ CFX_RetainPtr<IFX_FileAccess> GetNext(FX_POSITION& pos);
private:
CFX_ByteString GetNextFile();
« core/fxcrt/fx_extension.cpp ('K') | « core/fxcrt/fx_stream.h ('k') | xfa/fgas/font/cfgas_fontmgr.cpp » ('j') | xfa/fgas/font/cfgas_fontmgr.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698