Remove virtual keyboard support.

Remove virtual keyboard support:

1. common.gypi: use_virtual_keyboard define
2. Any code controlled by that define
3. chrome/browser/ui/virtual_keyboard/*
4. chrome/browser/resources/keyboard/*
5. experimental extension APIs for virtual keyboard

No user-visible changes.

BUG=128295
TEST=try

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

[hashimoto, 2012-05-17 05:00:06]

lgtm

[Yusuke Sato, 2012-05-17 05:12:42]

+bryeung

Bryan, I'm going to remove chrome.input APIs except
chrome.experimental.input.ime and sendKeyboardEvent. This means
chrome.experimental.input.ui.* will be removed. If you have any concern, please
let me know.

I'd keep the sendKeyboardEvent API since it's useful not only for VKs but also
for IMEs implemented on top of the chrome.experimental.input.ime API.

[bryeung, 2012-05-17 15:44:35]

+penghuang (as he wrote the chrome.experimental.input.ui.* APIs)

On Thu, May 17, 2012 at 1:12 AM,  <yusukes@chromium.org> wrote:
> +bryeung
>
> Bryan, I'm going to remove chrome.input APIs except
> chrome.experimental.input.ime and sendKeyboardEvent. This means
> chrome.experimental.input.ui.* will be removed. If you have any concern,
> please
> let me know.

This sounds okay to me.

> I'd keep the sendKeyboardEvent API since it's useful not only for VKs but
> also
> for IMEs implemented on top of the chrome.experimental.input.ime API.

Yes: I will probably need the sendKeyboardEvent API in the near future
for some work I'm doing.  Please leave it in.

Thanks,

Bryan

[Yusuke Sato, 2012-05-18 10:27:10]

Thanks Bryan.

+ben, +jar, +mpcomplete, +avayvod, +jhawkins
Could you do an OWNERS review for the cleanup patch?


On 2012/05/17 15:44:35, bryeung wrote:
> +penghuang (as he wrote the chrome.experimental.input.ui.* APIs)
> 
> On Thu, May 17, 2012 at 1:12 AM,  <mailto:yusukes@chromium.org> wrote:
> > +bryeung
> >
> > Bryan, I'm going to remove chrome.input APIs except
> > chrome.experimental.input.ime and sendKeyboardEvent. This means
> > chrome.experimental.input.ui.* will be removed. If you have any concern,
> > please
> > let me know.
> 
> This sounds okay to me.
> 
> > I'd keep the sendKeyboardEvent API since it's useful not only for VKs but
> > also
> > for IMEs implemented on top of the chrome.experimental.input.ime API.
> 
> Yes: I will probably need the sendKeyboardEvent API in the near future
> for some work I'm doing.  Please leave it in.
> 
> Thanks,
> 
> Bryan

[whywhat, 2012-05-18 10:34:18]

chromeos: lgtm

[Ben Goodger (Google), 2012-05-18 15:56:28]

Great! Thanks for cleaning this up :-)

LGTM

[Matt Perry, 2012-05-18 18:55:08]

lgtm

[Yusuke Sato, 2012-05-21 04:12:25]

jhawkins:
ping?

[James Hawkins, 2012-05-21 15:17:30]

LGTM.  This CL should have been broken up and submitted in pieces.

[Yusuke Sato, 2012-05-22 04:29:59]

On 2012/05/21 15:17:30, James Hawkins wrote:
> LGTM.  This CL should have been broken up and submitted in pieces.

sure, I'll do that next time..

[commit-bot: I haz the power, 2012-05-22 04:31:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yusukes@chromium.org/10399046/19003

[commit-bot: I haz the power, 2012-05-22 04:32:08]

Presubmit check for 10399046-19003 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit Messages **
See the JavaScript style guide at
http://www.chromium.org/developers/web-development-style-guide#TOC-JavaScript
and if you have any feedback about the JavaScript PRESUBMIT check, contact
tbreisacher@chromium.org

** Presubmit Warnings **
Found JavaScript style violations in
chrome/common/extensions/docs/js/api_page_generator.js:
  line 171: Use $('id'), from chrome://resources/js/util.js, instead of
document.getElementById('id'))
    document.getElementById('static').innerHTML = overviewContent;
    ^^^^^^^^^^^^^^^^^^^^^^^
  line 85: E0010: Missing semicolon at end of line
]
^
  line 128: E0220: No docs found for member 'Array.prototype.select'
Array.prototype.select = function(f) {
^^^^^^^^^^^^^^^^^^^^^^
  line 133: E0011: Missing semicolon after function assigned to a variable
}
^
  line 239: E0210: Missing docs for parameter: "url"
function fetchContent(url, onSuccess, onError) {
                                             ^^
  line 239: E0210: Missing docs for parameter: "onSuccess"
function fetchContent(url, onSuccess, onError) {
                                             ^^
  line 239: E0210: Missing docs for parameter: "onError"
function fetchContent(url, onSuccess, onError) {
                                             ^^
  line 258: E0002: Missing space before "{"
    xhr.onreadystatechange = function(){
                                       ^
  line 273: E0002: Missing space before "("
  } catch(e) {
         ^
  line 363: E0010: Missing semicolon at end of line
    var n = nodes[i]
                   ^
  line 409: E0002: Missing space before "("
  while(n = results.iterateNext()) {
       ^

Presubmit checks took 8.0s to calculate.

[Yusuke Sato, 2012-05-22 05:50:44]

the presubmit error is not related to this CL. manually submitting the change..

On 2012/05/22 04:32:08, I haz the power (commit-bot) wrote:
> Presubmit check for 10399046-19003 failed and returned exit status 1.
> 
> Running presubmit commit checks ...
> 
> ** Presubmit Messages **
> See the JavaScript style guide at
> http://www.chromium.org/developers/web-development-style-guide#TOC-JavaScript
> and if you have any feedback about the JavaScript PRESUBMIT check, contact
> mailto:tbreisacher@chromium.org
> 
> ** Presubmit Warnings **
> Found JavaScript style violations in
> chrome/common/extensions/docs/js/api_page_generator.js:
>   line 171: Use $('id'), from chrome://resources/js/util.js, instead of
> document.getElementById('id'))
>     document.getElementById('static').innerHTML = overviewContent;
>     ^^^^^^^^^^^^^^^^^^^^^^^
>   line 85: E0010: Missing semicolon at end of line
> ]
> ^
>   line 128: E0220: No docs found for member 'Array.prototype.select'
> Array.prototype.select = function(f) {
> ^^^^^^^^^^^^^^^^^^^^^^
>   line 133: E0011: Missing semicolon after function assigned to a variable
> }
> ^
>   line 239: E0210: Missing docs for parameter: "url"
> function fetchContent(url, onSuccess, onError) {
>                                              ^^
>   line 239: E0210: Missing docs for parameter: "onSuccess"
> function fetchContent(url, onSuccess, onError) {
>                                              ^^
>   line 239: E0210: Missing docs for parameter: "onError"
> function fetchContent(url, onSuccess, onError) {
>                                              ^^
>   line 258: E0002: Missing space before "{"
>     xhr.onreadystatechange = function(){
>                                        ^
>   line 273: E0002: Missing space before "("
>   } catch(e) {
>          ^
>   line 363: E0010: Missing semicolon at end of line
>     var n = nodes[i]
>                    ^
>   line 409: E0002: Missing space before "("
>   while(n = results.iterateNext()) {
>        ^
> 
> Presubmit checks took 8.0s to calculate.