more logs for 26646

more logs for 26646

BUG=chromium-os:26646
TEST=none

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

[Jorge Lucangeli Obes, 2012-03-02 19:15:57]

https://chromiumcodereview.appspot.com/9582019/diff/1/chrome/browser/automati...
File chrome/browser/automation/automation_provider.cc (right):

https://chromiumcodereview.appspot.com/9582019/diff/1/chrome/browser/automati...
chrome/browser/automation/automation_provider.cc:266: VLOG(2) <<
"SetExpectedTabCount:" << expected_tabs;
No #if defined(OS_CHROMEOS)?

https://chromiumcodereview.appspot.com/9582019/diff/1/chrome/browser/automati...
File chrome/browser/automation/automation_provider_observers.cc (right):

https://chromiumcodereview.appspot.com/9582019/diff/1/chrome/browser/automati...
chrome/browser/automation/automation_provider_observers.cc:121: #if
defined(OS_CHROMELOS)
OS_CHROMELOS->OS_CHROMEOS

[oshima, 2012-03-02 19:21:49]

https://chromiumcodereview.appspot.com/9582019/diff/1/chrome/browser/automati...
File chrome/browser/automation/automation_provider.cc (right):

https://chromiumcodereview.appspot.com/9582019/diff/1/chrome/browser/automati...
chrome/browser/automation/automation_provider.cc:266: VLOG(2) <<
"SetExpectedTabCount:" << expected_tabs;
On 2012/03/02 19:15:57, Jorge Lucangeli Obes wrote:
> No #if defined(OS_CHROMEOS)?

I'm using OS_CHROMEOS for LOG(ERROR) which doesn't require changes to chromeos
side. I'm using LOG(ERROR) in the file that I'm not sure VLOG(2) works without
command line change.

https://chromiumcodereview.appspot.com/9582019/diff/1/chrome/browser/automati...
File chrome/browser/automation/automation_provider_observers.cc (right):

https://chromiumcodereview.appspot.com/9582019/diff/1/chrome/browser/automati...
chrome/browser/automation/automation_provider_observers.cc:121: #if
defined(OS_CHROMELOS)
On 2012/03/02 19:15:57, Jorge Lucangeli Obes wrote:
> OS_CHROMELOS->OS_CHROMEOS

Doh! Thank you for catch. Fixed.

[Jorge Lucangeli Obes, 2012-03-02 19:23:22]

lgtm

[commit-bot: I haz the power, 2012-03-02 19:46:37]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Jorge Lucangeli Obes, 2012-03-02 19:49:15]

Oh right forgot about that! I'm only a full committer for Chrome OS.

[xiyuan, 2012-03-02 20:21:57]

lgtm

[commit-bot: I haz the power, 2012-03-02 20:22:28]

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