Remove screen dimensions from image sent with Feedback.

Remove screen dimensions from image sent with Feedback.
After speaking with the engineers on the feedback backend, I confirmed that the screen dimensions we send are ignored and instead the dimensions are pulled from the image itself. Hence this CL removes the dimensions and replaces them with a placeholder value (so that the protobuf to remain valid - without any dimensions, the server won't accept the report).

This accomplishes two things,
first, the hack we put in place to 'remember' the screen dimensions associated with a feedback report can now be removed.
second, not requiring a browser for dimenions makes it simple to deal with the case where feedback can be invoked without a browser (as referred to in the crbug for this CL).

R=asargent@chromium.org, jamescook@chromium.org, zork@chromium.org
BUG=297334
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=225275

[rkc, 2013-09-24 21:34:38]

jamescook@ for OWNERS review on chrome/browser/ui/ash/

[James Cook, 2013-09-24 21:36:20]

c/b/ui/ash lgtm

+pkotwicz fyi

[asargent_no_longer_on_chrome, 2013-09-24 23:22:56]

extensions parts LGTM

[Zachary Kuznia, 2013-09-24 23:46:43]

lgtm

https://codereview.chromium.org/24436002/diff/1/chrome/browser/feedback/feedb...
File chrome/browser/feedback/feedback_util.cc (right):

https://codereview.chromium.org/24436002/diff/1/chrome/browser/feedback/feedb...
chrome/browser/feedback/feedback_util.cc:227: const std::string&
description_template,
nit: fix this indent while you're here.

[rkc, 2013-09-25 00:00:35]

https://codereview.chromium.org/24436002/diff/1/chrome/browser/feedback/feedb...
File chrome/browser/feedback/feedback_util.cc (right):

https://codereview.chromium.org/24436002/diff/1/chrome/browser/feedback/feedb...
chrome/browser/feedback/feedback_util.cc:227: const std::string&
description_template,
On 2013/09/24 23:46:43, Zachary Kuznia wrote:
> nit: fix this indent while you're here.

Done.

[commit-bot: I haz the power, 2013-09-25 00:13:03]

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

[commit-bot: I haz the power, 2013-09-25 02:29:38]

Retried try job too often on linux_rel for step(s) content_browsertests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2013-09-25 18:45:46]

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

[commit-bot: I haz the power, 2013-09-25 21:48:00]

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

[rkc, 2013-09-25 23:14:48]

Committed patchset #2 manually as r225275 (presubmit successful).