Add Diagnose Errors button to navigation error page

Add Diagnose Errors button to navigation error page

BUG=254349
R=dpolukhin@chromium.org, thakis@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=214016

[Zachary Kuznia, 2013-07-26 00:28:12]

Please take a look.

[Dmitry Polukhin, 2013-07-26 00:31:55]

lgtm

[Dmitry Polukhin, 2013-07-26 00:33:41]

https://codereview.chromium.org/20604002/diff/1/chrome/renderer/resources/net...
File chrome/renderer/resources/neterror.html (right):

https://codereview.chromium.org/20604002/diff/1/chrome/renderer/resources/net...
chrome/renderer/resources/neterror.html:346: <button id="diagnose-button"
onclick="diagnoseErrors()"
Please show this button only in case of Chrome OS.

[Zachary Kuznia, 2013-07-26 00:54:41]

https://codereview.chromium.org/20604002/diff/1/chrome/renderer/resources/net...
File chrome/renderer/resources/neterror.html (right):

https://codereview.chromium.org/20604002/diff/1/chrome/renderer/resources/net...
chrome/renderer/resources/neterror.html:346: <button id="diagnose-button"
onclick="diagnoseErrors()"
On 2013/07/26 00:33:41, Dmitry Polukhin wrote:
> Please show this button only in case of Chrome OS.

Done.

[Zachary Kuznia, 2013-07-26 00:57:43]

Hi Nico,

Could you do an OWNERS review of this?

Thanks,
-Zach

[Dmitry Polukhin, 2013-07-26 01:00:20]

LGTM

[Nico, 2013-07-26 22:02:50]

code lgtm, but the bug is pretty light on the details: what's this extension,
why is this cros only, etc

[commit-bot: I haz the power, 2013-07-26 23:04:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/zork@chromium.org/20604002/15001

[Zachary Kuznia, 2013-07-27 01:05:17]

Committed patchset #4 manually as r214016 (presubmit successful).