Add pyauto tests and helper files for system-level testing of Web SQL Databases.

Add pyauto tests and helper files for system-level testing of Web SQL Databases.
BUG=59379
TEST=none

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

[Nirnimesh, 2010-11-01 06:37:29]

Is it possible to have a pyauto hook which can dump all info about all web
databases?

http://codereview.chromium.org/4223001/diff/4001/5002
File chrome/test/functional/databases.py (right):

http://codereview.chromium.org/4223001/diff/4001/5002#newcode24
chrome/test/functional/databases.py:24:
self.GetFileURLForDataPath('web_database/database_tester.html'))
'/' will not work on win

http://codereview.chromium.org/4223001/diff/4001/5002#newcode26
chrome/test/functional/databases.py:26: def _ParseResultFromTestPage(self,
json):
this only calls simlejson.loads(). Not sure if this worth it

http://codereview.chromium.org/4223001/diff/4001/5002#newcode43
chrome/test/functional/databases.py:43: """Helper function that checks that the
message sent from the test page
plz mention which test page

http://codereview.chromium.org/4223001/diff/4001/5002#newcode234
chrome/test/functional/databases.py:234: except:
Use the right exception, not all

http://codereview.chromium.org/4223001/diff/4001/5002#newcode256
chrome/test/functional/databases.py:256: except:
same here

http://codereview.chromium.org/4223001/diff/4001/5002#newcode259
chrome/test/functional/databases.py:259: 
All functions in this script are very specific the to html file added. How do we
automate real webpages which use html5 storage?

http://codereview.chromium.org/4223001/diff/4001/5004
File chrome/test/pyautolib/pyauto.py (right):

http://codereview.chromium.org/4223001/diff/4001/5004#newcode1629
chrome/test/pyautolib/pyauto.py:1629: js = '%s(%s)' % (function, ',
'.join(converted_args))
please add a logging.debug() here

[kkania, 2010-11-01 18:50:23]

http://codereview.chromium.org/4223001/diff/4001/5002
File chrome/test/functional/databases.py (right):

http://codereview.chromium.org/4223001/diff/4001/5002#newcode24
chrome/test/functional/databases.py:24:
self.GetFileURLForDataPath('web_database/database_tester.html'))
On 2010/11/01 06:37:29, Nirnimesh wrote:
> '/' will not work on win

I did the same thing for notifications.py. Do I need to change that too?  I can
open a file on python windows using /. What's the reason this won't work?

http://codereview.chromium.org/4223001/diff/4001/5002#newcode26
chrome/test/functional/databases.py:26: def _ParseResultFromTestPage(self,
json):
On 2010/11/01 06:37:29, Nirnimesh wrote:
> this only calls simlejson.loads(). Not sure if this worth it

Combined this with func below

http://codereview.chromium.org/4223001/diff/4001/5002#newcode43
chrome/test/functional/databases.py:43: """Helper function that checks that the
message sent from the test page
On 2010/11/01 06:37:29, Nirnimesh wrote:
> plz mention which test page

Done.

http://codereview.chromium.org/4223001/diff/4001/5002#newcode234
chrome/test/functional/databases.py:234: except:
On 2010/11/01 06:37:29, Nirnimesh wrote:
> Use the right exception, not all

Done.

http://codereview.chromium.org/4223001/diff/4001/5002#newcode256
chrome/test/functional/databases.py:256: except:
On 2010/11/01 06:37:29, Nirnimesh wrote:
> same here

Done.

http://codereview.chromium.org/4223001/diff/4001/5002#newcode259
chrome/test/functional/databases.py:259: 
On 2010/11/01 06:37:29, Nirnimesh wrote:
> All functions in this script are very specific the to html file added. How do
we
> automate real webpages which use html5 storage?

javascript

http://codereview.chromium.org/4223001/diff/4001/5004
File chrome/test/pyautolib/pyauto.py (right):

http://codereview.chromium.org/4223001/diff/4001/5004#newcode1629
chrome/test/pyautolib/pyauto.py:1629: js = '%s(%s)' % (function, ',
'.join(converted_args))
On 2010/11/01 06:37:29, Nirnimesh wrote:
> please add a logging.debug() here

Done.

[Nirnimesh, 2010-11-01 18:56:38]

LGTM

http://codereview.chromium.org/4223001/diff/4001/5002
File chrome/test/functional/databases.py (right):

http://codereview.chromium.org/4223001/diff/4001/5002#newcode24
chrome/test/functional/databases.py:24:
self.GetFileURLForDataPath('web_database/database_tester.html'))
On 2010/11/01 18:50:23, kkania wrote:
> On 2010/11/01 06:37:29, Nirnimesh wrote:
> > '/' will not work on win
> 
> I did the same thing for notifications.py. Do I need to change that too?  I
can
> open a file on python windows using /. What's the reason this won't work?

Yes, please fix that too
While python recognizes / and converts to \ behind the scenes, it's still better
to use os.path.join() instead. Ex: if you invoke a shell command using this
path, it'll fail