Wrap ChromeOS specific functions with OS_CHROMEOS macro

Wrap ChromeOS specific functions with OS_CHROMEOS macro

This patch is separated from http://codereview.chromium.org/10920091/

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

[nhiroki, 2012-09-10 07:08:07]

Hi, kochi-san and yoshiki-san

Can you review this? Thanks!

[yoshiki, 2012-09-10 07:13:36]

LGTM

[yoshiki, 2012-09-10 07:18:51]

Some of other files still include chromeos-specific headers. 

For example,
- chrome/browser/google_apis/drive_api_operations.h
- chrome/browser/google_apis/auth_service.cc
- chrome/browser/google_apis/drive_api_service.cc
include the header files under chrome/browser/chromeos/.

Could you add "defined(OS_CHEROMEOS)" as well? Or if impossible, please add TODO
comment.

[kochi, 2012-09-10 07:29:43]

Yoshiki, is this comment for http://codereview.chromium.org/10920091/ ?

On 2012/09/10 07:18:51, yoshiki wrote:
> Some of other files still include chromeos-specific headers. 
> 
> For example,
> - chrome/browser/google_apis/drive_api_operations.h
> - chrome/browser/google_apis/auth_service.cc
> - chrome/browser/google_apis/drive_api_service.cc
> include the header files under chrome/browser/chromeos/.
> 
> Could you add "defined(OS_CHEROMEOS)" as well? Or if impossible, please add
TODO
> comment.

[yoshiki, 2012-09-10 07:36:30]

Thanks, kochi-san. I added (almost) same comment as
http://codereview.chromium.org/10920091/#msg31/.

On 2012/09/10 07:29:43, Takayoshi Kochi wrote:
> Yoshiki, is this comment for http://codereview.chromium.org/10920091/ ?
> 
> On 2012/09/10 07:18:51, yoshiki wrote:
> > Some of other files still include chromeos-specific headers. 
> > 
> > For example,
> > - chrome/browser/google_apis/drive_api_operations.h
> > - chrome/browser/google_apis/auth_service.cc
> > - chrome/browser/google_apis/drive_api_service.cc
> > include the header files under chrome/browser/chromeos/.
> > 
> > Could you add "defined(OS_CHEROMEOS)" as well? Or if impossible, please add
> TODO
> > comment.

[commit-bot: I haz the power, 2012-09-10 07:41:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nhiroki@chromium.org/10914173/1

[commit-bot: I haz the power, 2012-09-10 09:27:34]

Change committed as 155690