Provide correct extra info for ChromeOS

Provide extra info form ARM

BUG=99190
TEST=ARM bots build


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

[stevenjb, 2011-10-15 01:57:31]

Fixing the ordering so that ChromeOS gets the right extra info.

[cpu_(ooo_6.6-7.5), 2011-10-17 01:11:44]

http://codereview.chromium.org/8139038/diff/5001/chrome/browser/component_upd...
File chrome/browser/component_updater/component_updater_configurator.cc (left):

http://codereview.chromium.org/8139038/diff/5001/chrome/browser/component_upd...
chrome/browser/component_updater/component_updater_configurator.cc:69: #else
but I thought that chromeos also defines OS_LINUX so it seems it enters at 59
and never hits 67

[stevenjb, 2011-10-17 17:26:11]

http://codereview.chromium.org/8139038/diff/5001/chrome/browser/component_upd...
File chrome/browser/component_updater/component_updater_configurator.cc (left):

http://codereview.chromium.org/8139038/diff/5001/chrome/browser/component_upd...
chrome/browser/component_updater/component_updater_configurator.cc:69: #else
On 2011/10/17 01:11:44, cpu wrote:
> but I thought that chromeos also defines OS_LINUX so it seems it enters at 59
> and never hits 67

That's correct, OS_CHROMEOS also defins OS_LINUX. The old code would only ever
enter the OS_LINUX section and never hit the OS_CHROMEOS section. This change is
to put OS_CHROMEOS first so that it does get entered instead of OS_LINUX.

[stevenjb, 2011-10-19 00:05:37]

ping?

[cpu_(ooo_6.6-7.5), 2011-10-19 17:34:52]

lgtm

[cpu_(ooo_6.6-7.5), 2011-10-19 17:35:41]

also note http://codereview.chromium.org/8344001/

[stevenjb, 2011-10-19 17:51:39]

Thanks, and thanks for the heads-up.

[commit-bot: I haz the power, 2011-10-19 17:51:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/stevenjb@chromium.org/8139038/5001

[commit-bot: I haz the power, 2011-10-19 19:20:29]

Change committed as 106343