[chromedriver] Separate stuff of chrome from chromedriver.

[chromedriver] Separate stuff of chrome from chromedriver.
Do some refactoring.
Move all stuff about chrome into a sub directory.
Also, delete file chrome_launcher_impl.cc.

NOTRY=true

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

[chrisgao (Use stgao instead), 2013-03-14 22:59:04]

Did some refactoring.
ptal

[kkania, 2013-03-14 23:13:04]

update the README.txt, so that people can understand the distinction

https://codereview.chromium.org/12848005/diff/1/chrome/chrome_tests.gypi
File chrome/chrome_tests.gypi (right):

https://codereview.chromium.org/12848005/diff/1/chrome/chrome_tests.gypi#newc...
chrome/chrome_tests.gypi:733: '../ui/ui.gyp:ui',
you don't need this, do you?

https://codereview.chromium.org/12848005/diff/1/chrome/chrome_tests.gypi#newc...
chrome/chrome_tests.gypi:999:
'test/chromedriver/chrome/web_view_impl_unittest.cc',
looks like this is out of order

[chrisgao (Use stgao instead), 2013-03-15 00:26:31]

ptal

https://codereview.chromium.org/12848005/diff/1/chrome/chrome_tests.gypi
File chrome/chrome_tests.gypi (right):

https://codereview.chromium.org/12848005/diff/1/chrome/chrome_tests.gypi#newc...
chrome/chrome_tests.gypi:733: '../ui/ui.gyp:ui',
On 2013/03/14 23:13:05, kkania wrote:
> you don't need this, do you?
Yes, we still need it. ui::KeyboardCode is used in ui_events.h which is included
by web_view.h.

Will remove it after updating ui_events.h in later CL.

https://codereview.chromium.org/12848005/diff/1/chrome/chrome_tests.gypi#newc...
chrome/chrome_tests.gypi:999:
'test/chromedriver/chrome/web_view_impl_unittest.cc',
On 2013/03/14 23:13:05, kkania wrote:
> looks like this is out of order

Done.

[kkania, 2013-03-15 00:43:21]

lgtm if tests pass

[commit-bot: I haz the power, 2013-03-15 16:16:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/chrisgao@chromium.org/12848005/7001

[commit-bot: I haz the power, 2013-03-15 16:20:04]

Change committed as 188410

[frankf, 2013-03-15 17:27:21]

On 2013/03/15 16:20:04, I haz the power (commit-bot) wrote:
> Change committed as 188410

This caused failures on Android:

http://build.chromium.org/p/chromium.fyi/builders/Android%20ChromeDriver%20Te...

[frankf, 2013-03-15 17:33:55]

Also, the path to version.cc needs to be udpated:

http://chromegw/i/chromium.pyauto/builders/Linux%20QA/builds/1557/steps/chrom...

On 2013/03/15 17:27:21, frankf wrote:
> On 2013/03/15 16:20:04, I haz the power (commit-bot) wrote:
> > Change committed as 188410
> 
> This caused failures on Android:
> 
>
http://build.chromium.org/p/chromium.fyi/builders/Android%2520ChromeDriver%25...