Add API on host registration page.

Add API on host registration page.
Add test register form page.
Added extra debug logging.

Full cycle:
1. Register screen navigates to chrome://register (host page).
2. host page requests DOMUI for register form URL: getRegistrationUrl msg on DOM load event.
3. Once URL is received it is loaded in an iframe.
4. When register form is loaded it sends get_user_info msg via postMessage to host page.
5. host page requests DOMUI for system/user info: getUserInfo msg.
6. Once info is received host page sends that to an iframe with set_user_info msg.
7. Register form notifies host page on registration success/skip status with complete_registration msg.
8. Based on registration result host page navigates to cros://register/[success|skipped] so that Out of box wizard is notified.

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

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

[Nikita (slow), 2010-07-23 11:03:41]

Erik, please review JS/HTML part.

[whywhat, 2010-07-23 11:28:51]

LGTM for C++ part.

http://codereview.chromium.org/3058006/diff/5001/6004
File chrome/browser/dom_ui/register_page_ui.cc (right):

http://codereview.chromium.org/3058006/diff/5001/6004#newcode166
chrome/browser/dom_ui/register_page_ui.cc:166: std::string url =
WizardController::default_controller()->
const std::string& url?

[arv (Not doing code reviews), 2010-07-23 18:51:34]

What is you i18n plan?

http://codereview.chromium.org/3058006/diff/5001/6005
File chrome/browser/resources/host_registration_page.html (right):

http://codereview.chromium.org/3058006/diff/5001/6005#newcode51
chrome/browser/resources/host_registration_page.html:51: var msg = {type:
'set_user_info',
var msg = {
  type: ...
  domain: ...
  ...
};

http://codereview.chromium.org/3058006/diff/5001/6005#newcode60
chrome/browser/resources/host_registration_page.html:60: <div id="main">
main seems useless to me. Can you just put the iframe in the body?

http://codereview.chromium.org/3058006/diff/5001/6005#newcode61
chrome/browser/resources/host_registration_page.html:61: <iframe id="form"
width="100%" height="100%" frameborder="0"></iframe>
move presentation to CSS

http://codereview.chromium.org/3058006/diff/5001/6006
File chrome/test/data/register_form.html (right):

http://codereview.chromium.org/3058006/diff/5001/6006#newcode37
chrome/test/data/register_form.html:37: $('url').innerHTML = location.href;
Use textContent

http://codereview.chromium.org/3058006/diff/5001/6006#newcode50
chrome/test/data/register_form.html:50: $('info').innerHTML = 'OS: ' +
info.os_name + ', ' +
textContent

http://codereview.chromium.org/3058006/diff/5001/6006#newcode54
chrome/test/data/register_form.html:54: $('messageInfo').innerHTML = 'e.origin:
' + e.origin +
textContent

http://codereview.chromium.org/3058006/diff/5001/6006#newcode70
chrome/test/data/register_form.html:70: <div id="main" class="gradient">
Can you just set the gradient in #main in the style. Then you do not need the
class

http://codereview.chromium.org/3058006/diff/5001/6006#newcode77
chrome/test/data/register_form.html:77: <a href="#"
onClick="registration(true);">Register</a> |
Use <button> with style. See new_new_tab.css for how to make buttons look like
links

http://codereview.chromium.org/3058006/diff/5001/6006#newcode77
chrome/test/data/register_form.html:77: <a href="#"
onClick="registration(true);">Register</a> |
it is onclick (all lower case)

[Nikita (slow), 2010-07-26 14:38:22]

As for i18n plan:

Host registration page doesn't hold any content.
We'll be passing selected language to a registration form:
https://oem-partner.com/register?lang=selected_lang

http://codereview.chromium.org/3058006/diff/5001/6004
File chrome/browser/dom_ui/register_page_ui.cc (right):

http://codereview.chromium.org/3058006/diff/5001/6004#newcode166
chrome/browser/dom_ui/register_page_ui.cc:166: std::string url =
WizardController::default_controller()->
On 2010/07/23 11:28:52, whywhat wrote:
> const std::string& url?

Done.

http://codereview.chromium.org/3058006/diff/5001/6005
File chrome/browser/resources/host_registration_page.html (right):

http://codereview.chromium.org/3058006/diff/5001/6005#newcode51
chrome/browser/resources/host_registration_page.html:51: var msg = {type:
'set_user_info',
On 2010/07/23 18:51:34, arv wrote:
> var msg = {
>   type: ...
>   domain: ...
>   ...
> };

Done.

http://codereview.chromium.org/3058006/diff/5001/6005#newcode60
chrome/browser/resources/host_registration_page.html:60: <div id="main">
On 2010/07/23 18:51:34, arv wrote:
> main seems useless to me. Can you just put the iframe in the body?

Done.

http://codereview.chromium.org/3058006/diff/5001/6005#newcode61
chrome/browser/resources/host_registration_page.html:61: <iframe id="form"
width="100%" height="100%" frameborder="0"></iframe>
On 2010/07/23 18:51:34, arv wrote:
> move presentation to CSS

Done.

http://codereview.chromium.org/3058006/diff/5001/6006
File chrome/test/data/register_form.html (right):

http://codereview.chromium.org/3058006/diff/5001/6006#newcode37
chrome/test/data/register_form.html:37: $('url').innerHTML = location.href;
On 2010/07/23 18:51:34, arv wrote:
> Use textContent

Done.

http://codereview.chromium.org/3058006/diff/5001/6006#newcode50
chrome/test/data/register_form.html:50: $('info').innerHTML = 'OS: ' +
info.os_name + ', ' +
On 2010/07/23 18:51:34, arv wrote:
> textContent

Done.

http://codereview.chromium.org/3058006/diff/5001/6006#newcode54
chrome/test/data/register_form.html:54: $('messageInfo').innerHTML = 'e.origin:
' + e.origin +
On 2010/07/23 18:51:34, arv wrote:
> textContent

Done.

http://codereview.chromium.org/3058006/diff/5001/6006#newcode70
chrome/test/data/register_form.html:70: <div id="main" class="gradient">
On 2010/07/23 18:51:34, arv wrote:
> Can you just set the gradient in #main in the style. Then you do not need the
> class

Done.

http://codereview.chromium.org/3058006/diff/5001/6006#newcode77
chrome/test/data/register_form.html:77: <a href="#"
onClick="registration(true);">Register</a> |
On 2010/07/23 18:51:34, arv wrote:
> it is onclick (all lower case)

Done.

http://codereview.chromium.org/3058006/diff/5001/6006#newcode77
chrome/test/data/register_form.html:77: <a href="#"
onClick="registration(true);">Register</a> |
On 2010/07/23 18:51:34, arv wrote:
> Use <button> with style. See new_new_tab.css for how to make buttons look like
> links

In fact they should be buttons.

[arv (Not doing code reviews), 2010-07-26 22:44:38]

lgtm

http://codereview.chromium.org/3058006/diff/19002/21005
File chrome/browser/resources/host_registration_page.html (right):

http://codereview.chromium.org/3058006/diff/19002/21005#newcode14
chrome/browser/resources/host_registration_page.html:14: width: 100%;
No need for width when both left and right are set (same for height)

http://codereview.chromium.org/3058006/diff/19002/21006
File chrome/test/data/register_form.html (right):

http://codereview.chromium.org/3058006/diff/19002/21006#newcode14
chrome/test/data/register_form.html:14: padding: 10px 10px 10px 10px;
padding: 10px;

http://codereview.chromium.org/3058006/diff/19002/21006#newcode53
chrome/test/data/register_form.html:53: $('messageInfo').textContent =
'e.origin: ' + e.origin + ', e.data.domain: ' + e.data.domain;
long line

[Nikita (slow), 2010-07-27 08:51:37]

http://codereview.chromium.org/3058006/diff/19002/21005
File chrome/browser/resources/host_registration_page.html (right):

http://codereview.chromium.org/3058006/diff/19002/21005#newcode14
chrome/browser/resources/host_registration_page.html:14: width: 100%;
On 2010/07/26 22:44:38, arv wrote:
> No need for width when both left and right are set (same for height)

If I don't specify 100% for w/h then iframe takes only that space that is
required by it's content. 
I need it to fill 100% space of the host page.

http://codereview.chromium.org/3058006/diff/19002/21006
File chrome/test/data/register_form.html (right):

http://codereview.chromium.org/3058006/diff/19002/21006#newcode14
chrome/test/data/register_form.html:14: padding: 10px 10px 10px 10px;
On 2010/07/26 22:44:38, arv wrote:
> padding: 10px;

Done.

http://codereview.chromium.org/3058006/diff/19002/21006#newcode53
chrome/test/data/register_form.html:53: $('messageInfo').textContent =
'e.origin: ' + e.origin + ', e.data.domain: ' + e.data.domain;
On 2010/07/26 22:44:38, arv wrote:
> long line

Done.

[arv (Not doing code reviews), 2010-07-27 18:38:12]

On Tue, Jul 27, 2010 at 01:51,  <nkostylev@chromium.org> wrote:
>
> http://codereview.chromium.org/3058006/diff/19002/21005
> File chrome/browser/resources/host_registration_page.html (right):
>
> http://codereview.chromium.org/3058006/diff/19002/21005#newcode14
> chrome/browser/resources/host_registration_page.html:14: width: 100%;
> On 2010/07/26 22:44:38, arv wrote:
>>
>> No need for width when both left and right are set (same for height)
>
> If I don't specify 100% for w/h then iframe takes only that space that
> is required by it's content.
> I need it to fill 100% space of the host page.

You are right. Replaced content sucks. However, you can use left+width
and skip the right.

> http://codereview.chromium.org/3058006/diff/19002/21006
> File chrome/test/data/register_form.html (right):
>
> http://codereview.chromium.org/3058006/diff/19002/21006#newcode14
> chrome/test/data/register_form.html:14: padding: 10px 10px 10px 10px;
> On 2010/07/26 22:44:38, arv wrote:
>>
>> padding: 10px;
>
> Done.
>
> http://codereview.chromium.org/3058006/diff/19002/21006#newcode53
> chrome/test/data/register_form.html:53: $('messageInfo').textContent =
> 'e.origin: ' + e.origin + ', e.data.domain: ' + e.data.domain;
> On 2010/07/26 22:44:38, arv wrote:
>>
>> long line
>
> Done.
>
> http://codereview.chromium.org/3058006/show
>