move chrome/browser/ui/gtk/certificate_viewer.* to chrome/browser/ui, add it to

move chrome/browser/ui/gtk/certificate_viewer.* to chrome/browser/ui, add it to
the aura build, and delete the stubs (which were ifdef'd out in Windows)


BUG=102946
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=110364

[oshima, 2011-11-14 23:11:40]

http://codereview.chromium.org/8543001/diff/1/chrome/chrome_browser.gypi
File chrome/chrome_browser.gypi (right):

http://codereview.chromium.org/8543001/diff/1/chrome/chrome_browser.gypi#newc...
chrome/chrome_browser.gypi:4165: ['include',
'^browser/ui/certificate_dialogs.h'],
Do you need this? Looks like this is already included above.
I think what you need is to exclude this for mac and win.
Linux, chromeos needs this regardless of whether or not they're aura.

[benrg, 2011-11-15 00:26:15]

http://codereview.chromium.org/8543001/diff/1/chrome/chrome_browser.gypi
File chrome/chrome_browser.gypi (right):

http://codereview.chromium.org/8543001/diff/1/chrome/chrome_browser.gypi#newc...
chrome/chrome_browser.gypi:4165: ['include',
'^browser/ui/certificate_dialogs.h'],
You're right.

[oshima, 2011-11-15 15:34:25]

LGTM

You need LGTM from owners as well.

[sky, 2011-11-15 20:33:48]

LGTM

http://codereview.chromium.org/8543001/diff/4001/chrome/browser/ui/webui/opti...
File chrome/browser/ui/webui/options/certificate_manager_handler.cc (right):

http://codereview.chromium.org/8543001/diff/4001/chrome/browser/ui/webui/opti...
chrome/browser/ui/webui/options/certificate_manager_handler.cc:17: #include
"chrome/browser/ui/certificate_dialogs.h"
sort

[commit-bot: I haz the power, 2011-11-16 20:39:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/benrg@chromium.org/8543001/11001

[commit-bot: I haz the power, 2011-11-16 22:22:02]

Change committed as 110364