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

Unified Diff: chrome/renderer/spellchecker/hunspell_engine.cc

Issue 109273002: Convert base::MemoryMappedFile to use File instead of PlatformFile. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Posix GetSize Created 7 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
Index: chrome/renderer/spellchecker/hunspell_engine.cc
diff --git a/chrome/renderer/spellchecker/hunspell_engine.cc b/chrome/renderer/spellchecker/hunspell_engine.cc
index 74dd292fb1585046871656324c7079c0737eae32..c2cdd29f560f2567965120a686c6fe39cb09900e 100644
--- a/chrome/renderer/spellchecker/hunspell_engine.cc
+++ b/chrome/renderer/spellchecker/hunspell_engine.cc
@@ -61,7 +61,9 @@ void HunspellEngine::InitializeHunspell() {
bdict_file_.reset(new base::MemoryMappedFile);
- if (bdict_file_->Initialize(file_)) {
+ // TODO(rvargas): This object should not keep file_ after passing it to
+ // bdict_file_.
+ if (bdict_file_->Initialize(base::File(file_))) {
TimeTicks debug_start_time = base::Histogram::DebugNow();
hunspell_.reset(

Powered by Google App Engine
This is Rietveld 408576698