Disable native messaging in non-supported configurations

Disable native messaging in non-supported configurations.

Native Messaging API is useful only on desktop OSes, i.e. on Linux, OSX and 
Windows. Disable it in all other configurations. Also improved how we handle 
the case when native messaging is disabled is handled: now the app will get 
disconnect notification.

BUG=142915
TBR=thakis@chromium.org


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

[Matt Perry, 2013-02-11 22:44:54]

lgtm

https://codereview.chromium.org/12218099/diff/1/chrome/browser/extensions/api...
File chrome/browser/extensions/api/messaging/message_service.cc (right):

https://codereview.chromium.org/12218099/diff/1/chrome/browser/extensions/api...
chrome/browser/extensions/api/messaging/message_service.cc:235: #endif  //
!(defined(OS_WIN) || defined(OS_MAXOSX) || defined(OS_LINUX))
nit: These comments seem backwards to me. The endif comment is actually matches
the else branch's condition. If anything should be labeled with the else
branch's condition, it should be the else branch itself.

[Sergey Ulanov, 2013-02-13 01:15:25]

https://codereview.chromium.org/12218099/diff/1/chrome/browser/extensions/api...
File chrome/browser/extensions/api/messaging/message_service.cc (right):

https://codereview.chromium.org/12218099/diff/1/chrome/browser/extensions/api...
chrome/browser/extensions/api/messaging/message_service.cc:235: #endif  //
!(defined(OS_WIN) || defined(OS_MAXOSX) || defined(OS_LINUX))
On 2013/02/11 22:44:54, Matt Perry wrote:
> nit: These comments seem backwards to me. The endif comment is actually
matches
> the else branch's condition. If anything should be labeled with the else
> branch's condition, it should be the else branch itself.

Done.

[commit-bot: I haz the power, 2013-02-13 01:17:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sergeyu@chromium.org/12218099/6

[commit-bot: I haz the power, 2013-02-13 01:18:00]

Presubmit check for 12218099-6 failed and returned exit status 1.

INFO:root:Found 2 file(s).

Running presubmit commit checks ...
Running /b/commit-queue/workdir/chromium/PRESUBMIT.py
Running /b/commit-queue/workdir/chromium/chrome/PRESUBMIT.py
Running /b/commit-queue/workdir/chromium/chrome/browser/extensions/PRESUBMIT.py

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome

[commit-bot: I haz the power, 2013-02-13 01:41:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sergeyu@chromium.org/12218099/6

[commit-bot: I haz the power, 2013-02-13 11:53:44]

Change committed as 182197