[Md Feedback] Add initial data population for dialog.

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

+// 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.
+
+#include "chrome/browser/ui/webui/md_feedback/md_feedback_webui_message_handler.h"
+
+#include "base/bind.h"
+#include "base/values.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/browser_finder.h"
+#include "chrome/browser/ui/tabs/tab_strip_model.h"
+#include "chrome/browser/ui/webui/md_feedback/md_feedback_ui.h"
+
+namespace {
+
+// Message names.
+const char kRequestData[] = "requestData";
+
+// JS function names.
+const char kSetData[] = "feedback.ui.setData";
+
+GURL GetTargetTabUrl(int session_id, int index) {

[afakhry, 2016/08/03 23:56:01]

This is the same code as in chrome::ShowFeedbackPage(). There's no need for
duplication. You can pass the URL in your other CL to the controller and remove
this.

[apacible, 2016/08/11 18:50:33]

My plan is to remove this (and  from chrome::ShowFeedbackPage once we switch
over. I think it makes sense to have it in the handler to pull the data directly
there rather than passing it around if we don't need to. WDYT?

+  Browser* browser = chrome::FindBrowserWithID(session_id);
+  // Sanity checks.
+  if (!browser || index >= browser->tab_strip_model()->count())
+    return GURL();
+
+  if (index >= 0) {
+    content::WebContents* target_tab =
+        browser->tab_strip_model()->GetWebContentsAt(index);
+    if (target_tab)
+      return target_tab->GetURL();
+  }
+
+  return GURL();
+}
+
+}  // namespace
+
+MdFeedbackWebUIMessageHandler::MdFeedbackWebUIMessageHandler(
+    MdFeedbackUI* md_feedback_ui)
+  : md_feedback_ui_(md_feedback_ui) {
+  DCHECK(md_feedback_ui_);
+}
+
+MdFeedbackWebUIMessageHandler::~MdFeedbackWebUIMessageHandler() {
+}
+
+void MdFeedbackWebUIMessageHandler::RegisterMessages() {
+  web_ui()->RegisterMessageCallback(
+      kRequestData,
+      base::Bind(&MdFeedbackWebUIMessageHandler::OnRequestData,
+                 base::Unretained(this)));
+}
+
+void MdFeedbackWebUIMessageHandler::OnRequestData(const base::ListValue* args) {
+  DVLOG(1) << "OnRequestData";
+  base::DictionaryValue data;
+
+  // Do not launch feedback on an OTR profile.
+  // TODO(apacible): Handle multiple profiles on CrOS.
+  Profile* profile =
+      ProfileManager::GetLastUsedProfileAllowedByPolicy();
+  profile = profile->GetOriginalProfile();
+  DCHECK(profile);
+  data.SetString("email", profile->GetProfileUserName());
+
+  GURL page_url = GURL();
+  Browser* browser = chrome::FindBrowserWithProfile(profile);
+  if (browser) {
+    page_url = GetTargetTabUrl(browser->session_id().id(),
+                               browser->tab_strip_model()->active_index());
+  }
+
+  data.SetString("url", page_url.spec());
+
+  web_ui()->CallJavascriptFunctionUnsafe(kSetData, data);
+}