Use chrome.windows to open chrome://system from Feedback.

Use chrome.windows to open chrome://system from Feedback.
Use chrome.windows to create a unique window to show the user their system information when the System Information link is clicked on the feedback UI.

R=xiyuan@chromium.org
BUG=268681
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=223563

[xiyuan, 2013-09-16 23:53:45]

https://codereview.chromium.org/23463035/diff/1/chrome/browser/resources/feed...
File chrome/browser/resources/feedback/js/feedback.js (right):

https://codereview.chromium.org/23463035/diff/1/chrome/browser/resources/feed...
chrome/browser/resources/feedback/js/feedback.js:67:
chrome.app.window.current().show();
Is this necessary? Could you pass focused: true in your createdata to make it
happen?

[rkc, 2013-09-17 00:00:00]

https://codereview.chromium.org/23463035/diff/1/chrome/browser/resources/feed...
File chrome/browser/resources/feedback/js/feedback.js (right):

https://codereview.chromium.org/23463035/diff/1/chrome/browser/resources/feed...
chrome/browser/resources/feedback/js/feedback.js:67:
chrome.app.window.current().show();
On 2013/09/16 23:53:45, xiyuan wrote:
> Is this necessary? Could you pass focused: true in your createdata to make it
> happen?

Focused:true I believe is the default. Passing that also has the same result,
the new window comes on top of the feedback window.

I specifically want the new window to be right under my app window so the user
doesn't get confused about where his feedback report went to.

[xiyuan, 2013-09-17 01:20:22]

LGTM

Thanks for the explanation.

[commit-bot: I haz the power, 2013-09-17 01:25:09]

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

[commit-bot: I haz the power, 2013-09-17 07:50:10]

Change committed as 223563