Automated ui test porting + cleanup:...

Automated ui test porting + cleanup:

- Change POINTs to gfx::Point
- Get rid of 2 unused automation messages (the messages themselves are staying for now so we don't mess with the reference build)
-- add new automation messages to replace GetWindowHWND, which is not portable
- re-enable automated_ui_test_interactive_test (it seems to have been dropped when we converted to gyp)
- compile additional tests on linux (they don't pass, so they are disabled)
- stub out linux tab dragging automation implementation (browser side)
- delete various cruft

BUG=19758

[huanr, 2009-09-20 23:34:59]

I thought GetHWND is used by several tests including browser_uitest. It is not
the case any more? Not sure about reposition.

http://codereview.chromium.org/211033/diff/1011/2005
File chrome/test/automation/automation_messages_internal.h (right):

http://codereview.chromium.org/211033/diff/1011/2005#newcode281
Line 281: //   std::vector<gfx::Point> - the path of the drag in window
coordinate space;
<=80 chars.

http://codereview.chromium.org/211033/diff/1011/2005#newcode1076
Line 1076: #if defined(OS_LINUX) || defined(OS_MACOSX)
I assume you added it here to avoid inserting message in the middle for
Linux/Mac? If so can you explain that in the comments as well?

[Evan Stade, 2009-09-21 23:22:52]

you are right, some tests do need it. I changed those calls to GetHWND to
portable replacements. (try bots seem to be screwed up right now, so I'll wait
till they work again before continuing with this)

[Evan Stade, 2009-09-22 02:05:54]

got all green trybot results, take another look?

[huanr, 2009-09-22 03:42:24]

LGTM

http://codereview.chromium.org/211033/diff/4024/4028
File chrome/test/automation/browser_proxy.h (right):

http://codereview.chromium.org/211033/diff/4024/4028#newcode198
Line 198: // Terminates the browser session.
Would be helpful if adding comments saying this happens due to the user shuts
down the machine or logs off. We already have a function UITest::CloseBrowser().

[Evan Stade, 2009-09-22 19:42:01]

http://codereview.chromium.org/211033/diff/4024/4028
File chrome/test/automation/browser_proxy.h (right):

http://codereview.chromium.org/211033/diff/4024/4028#newcode198
Line 198: // Terminates the browser session.
On 2009/09/22 03:42:24, huanr wrote:
> Would be helpful if adding comments saying this happens due to the user shuts
> down the machine or logs off. We already have a function
UITest::CloseBrowser().

Done.