Various Feedback fixes.

Various Feedback fixes.

This CL fixes two issues.
. It brings up a scroll bar if the text in the feedback window doesn't fit in
  the space available on screen (this happens when the user sets the page zoom
  too 120% or above)
. Sets the focus to the description field when the feedback app starts.

R=xiyuan@chromium.org
BUG=295999, 330139
Committed: https://crrev.com/8b02d82fc496c87bba7607ad3f903ff573c73e38
Cr-Commit-Position: refs/heads/master@{#292530}

[xiyuan, 2014-08-28 21:47:55]

https://codereview.chromium.org/511393002/diff/1/chrome/browser/resources/fee...
File chrome/browser/resources/feedback/css/feedback.css (right):

https://codereview.chromium.org/511393002/diff/1/chrome/browser/resources/fee...
chrome/browser/resources/feedback/css/feedback.css:14: overflow-y: auto;
Do we need to worry about overflow-x?

[rkc, 2014-08-28 21:59:03]

https://codereview.chromium.org/511393002/diff/1/chrome/browser/resources/fee...
File chrome/browser/resources/feedback/css/feedback.css (right):

https://codereview.chromium.org/511393002/diff/1/chrome/browser/resources/fee...
chrome/browser/resources/feedback/css/feedback.css:14: overflow-y: auto;
On 2014/08/28 21:47:55, xiyuan wrote:
> Do we need to worry about overflow-x?


It doesn't seem to help with the send button, but it does help in reaching the X
button on the titlebar at the top, so added it.

[xiyuan, 2014-08-28 22:13:04]

lgtm

https://codereview.chromium.org/511393002/diff/20001/chrome/browser/resources...
File chrome/browser/resources/feedback/css/feedback.css (right):

https://codereview.chromium.org/511393002/diff/20001/chrome/browser/resources...
chrome/browser/resources/feedback/css/feedback.css:15: overflow-y: auto;
nit: simplify to "overflow: auto"

[rkc, 2014-08-28 22:16:19]

https://codereview.chromium.org/511393002/diff/20001/chrome/browser/resources...
File chrome/browser/resources/feedback/css/feedback.css (right):

https://codereview.chromium.org/511393002/diff/20001/chrome/browser/resources...
chrome/browser/resources/feedback/css/feedback.css:15: overflow-y: auto;
On 2014/08/28 22:13:04, xiyuan wrote:
> nit: simplify to "overflow: auto"

Done.

[rkc, 2014-08-28 22:16:23]

The CQ bit was checked by rkc@chromium.org

[commit-bot: I haz the power, 2014-08-28 22:17:59]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rkc@chromium.org/511393002/40001

[commit-bot: I haz the power, 2014-08-29 00:01:49]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  ios_dbg_simulator on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  win_chromium_compile_dbg on tryserver.chromium.win
(http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[commit-bot: I haz the power, 2014-08-29 01:02:52]

Committed patchset #3 (id:40001) as 154e655127e27c5df9555692f2df8f744513aee8

[commit-bot: I haz the power, 2014-09-10 03:04:38]

Patchset 3 (id:??) landed as
https://crrev.com/8b02d82fc496c87bba7607ad3f903ff573c73e38
Cr-Commit-Position: refs/heads/master@{#292530}