[Md Feedback] Add initial data population for dialog.

chrome/browser/ui/webui/md_feedback/md_feedback_webui_message_handler.h

+// Copyright 2016 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_UI_WEBUI_MD_FEEDBACK_MD_FEEDBACK_WEBUI_MESSAGE_HANDLER_H_
+#define CHROME_BROWSER_UI_WEBUI_MD_FEEDBACK_MD_FEEDBACK_WEBUI_MESSAGE_HANDLER_H_
+
+#include "base/macros.h"
+#include "content/public/browser/web_ui_message_handler.h"
+
+namespace content {
+class WebUI;
+}

[afakhry, 2016/08/11 23:31:31]

We don't need this forward declare here. Please remove.

[apacible, 2016/08/12 01:46:42]

Done.

+
+class MdFeedbackUI;
+
+// The handler for Javascript messages related to the feedback dialog.
+class MdFeedbackWebUIMessageHandler : public content::WebUIMessageHandler {
+ public:
+  explicit MdFeedbackWebUIMessageHandler(MdFeedbackUI* md_feedback_ui);
+  ~MdFeedbackWebUIMessageHandler() override;
+
+ private:
+  // WebUIMessageHandler implementation.

[afakhry, 2016/08/11 23:31:31]

Recently this parent-class override comment has been shortened to just:

// WebUIMessageHandler:

[apacible, 2016/08/12 01:46:42]

Done.

+  void RegisterMessages() override;
+
+  // Handlers for JavaScript messages.

[afakhry, 2016/08/11 23:31:32]

Hmmm, I'd rather this comment tells us what OnRequestData() actually does.

[apacible, 2016/08/12 01:46:42]

Done.

+  void OnRequestData(const base::ListValue* args);
+
+  MdFeedbackUI* md_feedback_ui_;

[afakhry, 2016/08/11 23:31:32]

Nit [optional]:
MdFeedbackUI* md_feedback_ui_;  // Not owned.

[apacible, 2016/08/12 01:46:42]

Done.

+
+  DISALLOW_COPY_AND_ASSIGN(MdFeedbackWebUIMessageHandler);
+};
+
+#endif  // CHROME_BROWSER_UI_WEBUI_MD_FEEDBACK_MD_FEEDBACK_WEBUI_MESSAGE_HANDLER_H_