Fix file uploads in chromedriver. It should not be assumed that multiple file

Fix file uploads in chromedriver. It should not be assumed that multiple file
names are separate elements in the input array.
BUG=92911, 93909
TEST=none


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

[Huyen, 2011-08-23 22:10:44]

http://codereview.chromium.org/7701010/diff/3001/chrome/test/webdriver/comman...
File chrome/test/webdriver/commands/webelement_commands.cc (right):

http://codereview.chromium.org/7701010/diff/3001/chrome/test/webdriver/comman...
chrome/test/webdriver/commands/webelement_commands.cc:553:
base::SplitString(paths_string, '\n', &paths);
why not just directly go from array to vector of string?

[kkania, 2011-08-25 20:16:58]

http://codereview.chromium.org/7701010/diff/3001/chrome/test/webdriver/comman...
File chrome/test/webdriver/commands/webelement_commands.cc (right):

http://codereview.chromium.org/7701010/diff/3001/chrome/test/webdriver/comman...
chrome/test/webdriver/commands/webelement_commands.cc:553:
base::SplitString(paths_string, '\n', &paths);
not sure what you mean:

The input is an array of characters or strings, e.g. ['h', 'e','l','l','o'] or
['he', 'llo']. (the spec isn't clear, although i think really the input should
be single chars

I need to convert those characters to separate paths. I need to deal with extra
trailing whitespace, which SplitString does for me. Can you think of an easier
way to do this then what I have above?

[kkania, 2011-08-26 17:41:19]

ping

[Huyen, 2011-08-26 18:27:04]

On 2011/08/26 17:41:19, kkania wrote:
> ping

lgtm

[commit-bot: I haz the power, 2011-08-30 09:35:30]

Change committed as 98796