Add spellcheck and word suggestion to the prediction service

services/prediction/dictionary_service.h

+// Copyright 2015 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 SERVICES_PREDICTION_DICTIONARY_SERVICE_H_
+#define SERVICES_PREDICTION_DICTIONARY_SERVICE_H_
+
+#include "base/macros.h"
+#include "mojo/services/prediction/public/interfaces/prediction.mojom.h"
+#include "services/prediction/proximity_info_service.h"
+
+namespace latinime {
+
+class Dictionary;
+class DicTraverseSession;
+class PrevWordsInfo;
+}

[APW, 2015/07/23 18:02:15]

// namespace latinime

[riajiang, 2015/07/31 02:13:04]

Done.

+
+namespace prediction {
+
+class DictionaryService {
+ public:
+  DictionaryService();
+  ~DictionaryService();
+
+  mojo::Array<mojo::String> GetDictionarySuggestion(
+      PredictionInfoPtr prediction_info,
+      ProximityInfoService* proximity_info);
+
+ private:
+  int max_suggestion_size;

[APW, 2015/07/23 18:02:15]

private member variables must end in underscore.

so in this case: max_suggestion_size_

Check this everywhere.

Also, not sure if this is a style convention or not, but I've always seen member
functions above member variables in the header declarations.

[riajiang, 2015/07/31 02:13:04]

Done.

How about the sequence between static member variables and member functions? 

+  latinime::Dictionary* default_dictionary;
+  latinime::DicTraverseSession* default_session;
+
+  void LoadDictionary();
+
+  latinime::Dictionary* const OpenDictionary(const std::string path,
+                                             const int start_offset,
+                                             const int size,
+                                             const bool is_updatable);
+
+  latinime::PrevWordsInfo* ProcessPrevWord(
+      mojo::Array<mojo::String>& prev_words,
+      mojo::Array<bool>& beginnings);
+
+  DISALLOW_COPY_AND_ASSIGN(DictionaryService);
+};
+
+}  // namespace prediction
+
+#endif  // SERVICES_PREDICTION_DICTIONARY_SERVICE_H_