Add category selector to feedback dialog.

Add category selector to feedback dialog.

* Add a <select> element to the dialog allowing the user to choose a category, or "other".
* Disable the OK button and hide the reset options until a category is selected.
* Clean up some duplicated code now that this dialog depends on base.js.

BUG=b/20950552

Committed: https://crrev.com/1a814c9ea20387180c97adf639e47b9c4ae01392
Cr-Commit-Position: refs/heads/master@{#329784}

[Jamie, 2015-05-13 21:55:49]

jamiewalch@chromium.org changed reviewers:
+ kelvinp@chromium.org

[Jamie, 2015-05-13 21:55:49]

ptal

[kelvinp, 2015-05-13 22:40:04]

https://codereview.chromium.org/1131153003/diff/1/remoting/resources/remoting...
File remoting/resources/remoting_strings.grd (right):

https://codereview.chromium.org/1131153003/diff/1/remoting/resources/remoting...
remoting/resources/remoting_strings.grd:509: <message desc="Category indicating
that the user is having problems launching the application."
name="IDS_FEEDBACK_CATEGORY_APP_LAUNCH">
If the app can't open, the user probably won't have a chance to give feedback as
we only show the feedback context menu once the user is connected.

[Jamie, 2015-05-13 23:15:06]

https://codereview.chromium.org/1131153003/diff/1/remoting/resources/remoting...
File remoting/resources/remoting_strings.grd (right):

https://codereview.chromium.org/1131153003/diff/1/remoting/resources/remoting...
remoting/resources/remoting_strings.grd:509: <message desc="Category indicating
that the user is having problems launching the application."
name="IDS_FEEDBACK_CATEGORY_APP_LAUNCH">
On 2015/05/13 22:40:04, kelvinp wrote:
> If the app can't open, the user probably won't have a chance to give feedback
as
> we only show the feedback context menu once the user is connected.

On ChromeOS it's visible all the time, and I think we should switch to using a
top-level window on other platforms, at which point it can be visible all the
time on all platforms. Also, there are situations where the user could be
connected but the app doesn't launch, so I think it's worth keeping it as an
option.

[kelvinp, 2015-05-14 01:24:53]

lgtm

[Jamie, 2015-05-14 01:36:12]

The CQ bit was checked by jamiewalch@chromium.org

[commit-bot: I haz the power, 2015-05-14 01:36:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1131153003/1

[commit-bot: I haz the power, 2015-05-14 02:23:59]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-05-14 02:24:57]

Patchset 1 (id:??) landed as
https://crrev.com/1a814c9ea20387180c97adf639e47b9c4ae01392
Cr-Commit-Position: refs/heads/master@{#329784}