Register getUserInfo message callback for register screen.

Add getUserInfo, getRegistrationUrl message callback for register screen.
Restrict chrome://register to OOBE wizard only.
Actually delete about:register.

BUG= http://crosbug.com/4813
TEST=none

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

[Nikita (slow), 2010-07-16 17:15:05]

Now passing (some) system/user info to host registration page, registration form
url.
Next CL will focus on host registration page itself.

[whywhat, 2010-07-16 17:27:35]

LGTM

http://codereview.chromium.org/3044001/diff/6001/7002
File chrome/browser/chromeos/login/wizard_controller.cc (left):

http://codereview.chromium.org/3044001/diff/6001/7002#oldcode21
chrome/browser/chromeos/login/wizard_controller.cc:21: #include
"chrome/browser/chromeos/customization_document.h"
I'd leave this here if the pointer is used here too.

http://codereview.chromium.org/3044001/diff/6001/7003
File chrome/browser/chromeos/login/wizard_controller.h (right):

http://codereview.chromium.org/3044001/diff/6001/7003#newcode12
chrome/browser/chromeos/login/wizard_controller.h:12: #include
"chrome/browser/chromeos/customization_document.h"
Do we need this? I don't see any usage of the definition of the class, only
passing pointers back and forth.

http://codereview.chromium.org/3044001/diff/6001/7005
File chrome/browser/dom_ui/register_page_ui.cc (right):

http://codereview.chromium.org/3044001/diff/6001/7005#newcode30
chrome/browser/dom_ui/register_page_ui.cc:30: const wchar_t kJSONOsName[] =
L"ChromeOS";
Maybe kJsonOsName or kOSName?
L"ChromiumOS"?

http://codereview.chromium.org/3044001/diff/6001/7005#newcode102
chrome/browser/dom_ui/register_page_ui.cc:102: if
(!WizardController::default_controller()) {
This condition will be false also on login screen, not only during OOBE. Do we
have is_oobe public method for controller to use?

http://codereview.chromium.org/3044001/diff/6001/7006
File chrome/browser/resources/host_registration_page.html (right):

http://codereview.chromium.org/3044001/diff/6001/7006#newcode11
chrome/browser/resources/host_registration_page.html:11:
chrome.send('getRegistrationUrl', []);
Should we check chrome for being not null?

http://codereview.chromium.org/3044001/diff/6001/7006#newcode24
chrome/browser/resources/host_registration_page.html:24:
chrome.send('getUserInfo', []);
Ditto.

[arv (Not doing code reviews), 2010-07-16 17:47:28]

FYI

http://codereview.chromium.org/3044001/diff/6001/7006
File chrome/browser/resources/host_registration_page.html (right):

http://codereview.chromium.org/3044001/diff/6001/7006#newcode11
chrome/browser/resources/host_registration_page.html:11:
chrome.send('getRegistrationUrl', []);
On 2010/07/16 17:27:37, whywhat wrote:
> Should we check chrome for being not null?

No need to check that from a DOMUI

[Nikita (slow), 2010-07-19 11:46:24]

http://codereview.chromium.org/3044001/diff/6001/7002
File chrome/browser/chromeos/login/wizard_controller.cc (left):

http://codereview.chromium.org/3044001/diff/6001/7002#oldcode21
chrome/browser/chromeos/login/wizard_controller.cc:21: #include
"chrome/browser/chromeos/customization_document.h"
On 2010/07/16 17:27:37, whywhat wrote:
> I'd leave this here if the pointer is used here too.

Done.

http://codereview.chromium.org/3044001/diff/6001/7003
File chrome/browser/chromeos/login/wizard_controller.h (right):

http://codereview.chromium.org/3044001/diff/6001/7003#newcode12
chrome/browser/chromeos/login/wizard_controller.h:12: #include
"chrome/browser/chromeos/customization_document.h"
On 2010/07/16 17:27:37, whywhat wrote:
> Do we need this? I don't see any usage of the definition of the class, only
> passing pointers back and forth.

Done.

http://codereview.chromium.org/3044001/diff/6001/7005
File chrome/browser/dom_ui/register_page_ui.cc (right):

http://codereview.chromium.org/3044001/diff/6001/7005#newcode30
chrome/browser/dom_ui/register_page_ui.cc:30: const wchar_t kJSONOsName[] =
L"ChromeOS";
On 2010/07/16 17:27:37, whywhat wrote:
> Maybe kJsonOsName or kOSName?
> L"ChromiumOS"?

Done.

http://codereview.chromium.org/3044001/diff/6001/7005#newcode102
chrome/browser/dom_ui/register_page_ui.cc:102: if
(!WizardController::default_controller()) {
On 2010/07/16 17:27:37, whywhat wrote:
> This condition will be false also on login screen, not only during OOBE. Do we
> have is_oobe public method for controller to use?

Done.

http://codereview.chromium.org/3044001/diff/6001/7006
File chrome/browser/resources/host_registration_page.html (right):

http://codereview.chromium.org/3044001/diff/6001/7006#newcode24
chrome/browser/resources/host_registration_page.html:24:
chrome.send('getUserInfo', []);
On 2010/07/16 17:27:37, whywhat wrote:
> Ditto.

Not needed.