Local NTP: Introduce OneGoogleBarFetcher

chrome/browser/search/one_google_bar/one_google_bar_data.h

+// Copyright 2017 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 CHROME_BROWSER_SEARCH_ONE_GOOGLE_BAR_ONE_GOOGLE_BAR_DATA_H_
+#define CHROME_BROWSER_SEARCH_ONE_GOOGLE_BAR_ONE_GOOGLE_BAR_DATA_H_
+
+#include <string>
+
+struct OneGoogleBarData {
+  OneGoogleBarData();
+  OneGoogleBarData(const OneGoogleBarData&);
+  ~OneGoogleBarData();
+
+  std::string bar_html;
+
+  std::string in_head_script;
+  std::string in_head_noscript;

[sfiera, 2017/04/12 17:25:47]

Comments? I can guess what most of these are, but in_head_noscript lost me.

[Marc Treib, 2017/04/13 09:03:34]

It's for a <noscript> tag :)  https://www.w3schools.com/tags/tag_noscript.asp

However,
a) there are no plans to actually use it (the remote NTP doesn't, and it makes
no sense on the local one either), and
b) it seems to be empty in practice anyway,
so I've just removed it. I've added comments for the rest, though it's mostly
"this is what the API spits out".

+  std::string in_head_style;
+  std::string after_bar_script;
+  std::string end_of_body_html;
+  std::string end_of_body_script;
+
+  // Copying and assignment allowed.
+};
+
+#endif  // CHROME_BROWSER_SEARCH_ONE_GOOGLE_BAR_ONE_GOOGLE_BAR_DATA_H_