Make chromedriver test suites compatible with python2.7

Make chromedriver test suites compatible with python2.7
BUG = http://code.google.com/p/chromedriver/issues/detail?id=165
TEST = src/chrome/test/webdriver/test/run_webdriver_tests.py
       src/chrome/test/webdriver/test/run_chromedriver_tests.py
 
Contributed by yupingx.chen@intel.com


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

[Yang Gu, 2012-10-23 07:10:54]

The chromedriver test suites will throw AttributeError under python 2.7, please
see http://code.google.com/p/chromedriver/issues/detail?id=165 for details.

Update src/chrome/test/webdriver/test/py_unittest_util.py to fix the bug.

[kkania, 2012-10-23 15:03:22]

lgtm

[commit-bot: I haz the power, 2012-10-24 01:35:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yang.gu@intel.com/11236062/1

[commit-bot: I haz the power, 2012-10-24 01:36:02]

Presubmit check for 11236062-1 failed and returned exit status 1.


Running presubmit commit checks ...

** Presubmit Warnings **
yang.gu@intel.com is not in AUTHORS file. If you are a new contributor, please
visit
http://www.chromium.org/developers/contributing-code and read the "Legal"
section
If you are a chromite, verify the contributor signed the CLA.

[commit-bot: I haz the power, 2012-10-24 02:17:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yang.gu@intel.com/11236062/6002

[commit-bot: I haz the power, 2012-10-24 02:34:24]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yang.gu@intel.com/11236062/3003

[commit-bot: I haz the power, 2012-10-24 02:34:27]

Failed to apply patch for AUTHORS:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file AUTHORS
  Hunk #1 FAILED at 208.
  1 out of 1 hunk FAILED -- saving rejects to file AUTHORS.rej

Patch:       AUTHORS
Index: AUTHORS
diff --git a/AUTHORS b/AUTHORS
index
4ade4a636da3623657419fc2200104f7d7ac29d8..faae225af517f3cdb25335637b1f25ab41d610a8
100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -208,3 +208,4 @@ Joe Thomas <mhx348@motorola.com>
 Ruben Terrazas <rubentopo@gmail.com>
 Josh Triplett <josh.triplett@intel.com>
 Qiankun Miao <qiankun.miao@intel.com>
+Yang Gu <yang.gu@intel.com>

[commit-bot: I haz the power, 2012-10-24 02:36:20]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yang.gu@intel.com/11236062/3003

[commit-bot: I haz the power, 2012-10-24 02:36:23]

Failed to apply patch for AUTHORS:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file AUTHORS
  Hunk #1 FAILED at 208.
  1 out of 1 hunk FAILED -- saving rejects to file AUTHORS.rej

Patch:       AUTHORS
Index: AUTHORS
diff --git a/AUTHORS b/AUTHORS
index
4ade4a636da3623657419fc2200104f7d7ac29d8..faae225af517f3cdb25335637b1f25ab41d610a8
100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -208,3 +208,4 @@ Joe Thomas <mhx348@motorola.com>
 Ruben Terrazas <rubentopo@gmail.com>
 Josh Triplett <josh.triplett@intel.com>
 Qiankun Miao <qiankun.miao@intel.com>
+Yang Gu <yang.gu@intel.com>

[commit-bot: I haz the power, 2012-10-24 05:39:42]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yang.gu@intel.com/11236062/9006

[commit-bot: I haz the power, 2012-10-24 07:30:16]

Retried try job too often for step(s) browser_tests

[commit-bot: I haz the power, 2012-10-24 07:45:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yang.gu@intel.com/11236062/9006

[commit-bot: I haz the power, 2012-10-24 08:23:17]

Change committed as 163791

[kkania, 2012-10-24 15:02:01]

On 2012/10/24 08:23:17, I haz the power (commit-bot) wrote:
> Change committed as 163791

Ah, have you signed the CCLA
http://code.google.com/legal/corporate-cla-v1.0.html here?  You need to do that
and email it to cla-submissions@google.com.  Let me know.

[Yang Gu, 2012-10-25 01:59:10]

Yes, Intel helped me sign CLA with Google several months ago. I'm not sure how I
can provide some evidence on this. In fact, my account using Intel email address
was activated from then on. 

Regards,
-Yang

> -----Original Message-----
> From: kkania@chromium.org [mailto:kkania@chromium.org]
> Sent: Wednesday, October 24, 2012 11:02 PM
> To: Gu, Yang
> Cc: chromium-reviews@chromium.org; Chen, YupingX
> Subject: Re: Make chromedriver test suites compatible with python2.7 (issue
> 11236062)
> 
> On 2012/10/24 08:23:17, I haz the power (commit-bot) wrote:
> > Change committed as 163791
> 
> Ah, have you signed the CCLA
> http://code.google.com/legal/corporate-cla-v1.0.html here?  You need to do
that
> and email it to cla-submissions@google.com.  Let me know.
> 
> https://chromiumcodereview.appspot.com/11236062/