New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 632112 link

Starred by 2 users
Status: Archived
Owner:
█ apaci...@chromium.org
no longer active
Closed: Sep 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 2
Type: Bug



Sign in to add a comment

[MD Feedback] Wire up data messaging.

Project Member Reported by apaci...@chromium.org, Jul 27 2016 
Wire up JS<->C++ data messaging.

This includes*:
- Populating the feedback form (email, screenshot, sys info, metrics).
- Submission to feedback buckets.


* subject to change
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 15 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/dc2b3be0e417c19e4b2df35594a4ab5bec917dd4

commit dc2b3be0e417c19e4b2df35594a4ab5bec917dd4
Author: apacible <apacible@chromium.org>
Date: Thu Sep 15 21:15:03 2016

[Md Feedback] Add initial data population for dialog.

This change populates the feedback dialog with some initial info, including profile email and current URL, by adding a new WebUI Message Handler for C++ <-> JS messaging. The handler will be build out to support more messaging in the future, e.g. system info, screenshots, and feedback submission.

BUG= 632112 

Review-Url: https://codereview.chromium.org/2190653003
Cr-Commit-Position: refs/heads/master@{#418977}

[modify] https://crrev.com/dc2b3be0e417c19e4b2df35594a4ab5bec917dd4/chrome/browser/BUILD.gn
[modify] https://crrev.com/dc2b3be0e417c19e4b2df35594a4ab5bec917dd4/chrome/browser/browser_resources.grd
[add] https://crrev.com/dc2b3be0e417c19e4b2df35594a4ab5bec917dd4/chrome/browser/feedback/feedback_dialog_utils.cc
[add] https://crrev.com/dc2b3be0e417c19e4b2df35594a4ab5bec917dd4/chrome/browser/feedback/feedback_dialog_utils.h
[modify] https://crrev.com/dc2b3be0e417c19e4b2df35594a4ab5bec917dd4/chrome/browser/feedback/show_feedback_page.cc
[modify] https://crrev.com/dc2b3be0e417c19e4b2df35594a4ab5bec917dd4/chrome/browser/resources/md_feedback/feedback.html
[add] https://crrev.com/dc2b3be0e417c19e4b2df35594a4ab5bec917dd4/chrome/browser/resources/md_feedback/feedback.js
[modify] https://crrev.com/dc2b3be0e417c19e4b2df35594a4ab5bec917dd4/chrome/browser/resources/md_feedback/feedback_container.html
[modify] https://crrev.com/dc2b3be0e417c19e4b2df35594a4ab5bec917dd4/chrome/browser/resources/md_feedback/feedback_container.js
[modify] https://crrev.com/dc2b3be0e417c19e4b2df35594a4ab5bec917dd4/chrome/browser/ui/BUILD.gn
[modify] https://crrev.com/dc2b3be0e417c19e4b2df35594a4ab5bec917dd4/chrome/browser/ui/webui/md_feedback/md_feedback_ui.cc
[modify] https://crrev.com/dc2b3be0e417c19e4b2df35594a4ab5bec917dd4/chrome/browser/ui/webui/md_feedback/md_feedback_ui.h
[add] https://crrev.com/dc2b3be0e417c19e4b2df35594a4ab5bec917dd4/chrome/browser/ui/webui/md_feedback/md_feedback_webui_message_handler.cc
[add] https://crrev.com/dc2b3be0e417c19e4b2df35594a4ab5bec917dd4/chrome/browser/ui/webui/md_feedback/md_feedback_webui_message_handler.h

Comment 2 by apaci...@chromium.org, Sep 15 2017

Status: Archived (was: Started)
►

Sign in to add a comment