Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1324)

Unified Diff: chrome/browser/ui/webui/md_feedback/md_feedback_webui_message_handler.cc

Issue 2190653003: [Md Feedback] Add initial data population for dialog. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/md_feedback/md_feedback_webui_message_handler.cc
diff --git a/chrome/browser/ui/webui/md_feedback/md_feedback_webui_message_handler.cc b/chrome/browser/ui/webui/md_feedback/md_feedback_webui_message_handler.cc
new file mode 100644
index 0000000000000000000000000000000000000000..4ee46fa03ed0a9d31c16923d80ada531c5db91a2
--- /dev/null
+++ b/chrome/browser/ui/webui/md_feedback/md_feedback_webui_message_handler.cc
@@ -0,0 +1,80 @@
+// 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::ShowFeedbackPa
apacible 2016/08/11 18:50:33 My plan is to remove this (and from chrome::ShowF
+ 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);
+}

Powered by Google App Engine
This is Rietveld 408576698