Creating the NTP snippet service.

components/ntp_snippets/ntp_snippets_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 COMPONENTS_NTP_SNIPPETS_NTP_SNIPPETS_SERVICE_H_
+#define COMPONENTS_NTP_SNIPPETS_NTP_SNIPPETS_SERVICE_H_
+
+#include <string>
+
+#include "base/macros.h"
+#include "components/keyed_service/core/keyed_service.h"
+
+namespace ntp_snippets {
+
+// Stores and vend fresh content data for the NTP.
+class NTPSnippetsService : public KeyedService {
+ public:
+  // |application_language_code| should be a ISO 639-1 compliant string. Aka
+  // 'en' or 'en-US'. Note that this code should only specify the language, not
+  // the locale, so 'en_US' (english language with US locale) and 'en-GB_US'
+  // (British english person in the US) are not language code.
+  explicit NTPSnippetsService(const std::string& application_language_code);
+  void Shutdown() override;

[sdefresne, 2015/11/18 17:59:06]

style: the constructor and destructor must come before other methods, so the
order should be NTPSnippetsService(), ~ NTPSnippetsService(), Shutdown()

https://google.github.io/styleguide/cppguide.html#Declaration_Order

nit: the usage is generally to prefix method overrides from super-class by a
comment like this "// KeyedService implementation.". 

[noyau (Ping after 24h), 2015/11/19 09:08:21]

Again, this is the same order as in the superclass, and I think this is here for
a reason: showing that shutdown should be called before the destructor. If you
look at the superclass this was done on purpose, and I'm just following the lead
there. And I already explained this in a previous comment you made.

[sdefresne, 2015/11/19 11:34:43]

This does not seems to be what other classes inheriting from KeyedService do:
https://code.google.com/p/chromium/codesearch#search/&q=//%5C%20KeyedService&...

[sdefresne, 2015/11/19 11:46:49]

Sent https://codereview.chromium.org/1459843002 to fix the style guide violation
in the super class.

[noyau (Ping after 24h), 2015/11/19 12:36:14]

Done.

+  ~NTPSnippetsService() override;
+
+  // TODO(crbug.com/547046): Add API.
+ private:
+  // The ISO 639-1 code of the language used by the application.
+  const std::string application_language_code_;
+
+  DISALLOW_COPY_AND_ASSIGN(NTPSnippetsService);
+};
+
+}  // namespace ntp_snippets
+
+#endif  // COMPONENTS_NTP_SNIPPETS_NTP_SNIPPETS_SERVICE_H_