WindowsError is not cross platform, use OSError instead.

Thanks to user me.ssword for pointing this out. Fixes Issue 5034
nimeacuerdo · Jan 29, 2013 · 55a09d9 · 55a09d9
1 parent 12cdd1e
commit 55a09d9
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/py/selenium/webdriver/chrome/service.py b/py/selenium/webdriver/chrome/service.py
@@ -99,6 +99,6 @@ def stop(self):
             if self.process:
                 self.process.kill()
                 self.process.wait()
-        except WindowsError:
+        except OSError:
             # kill may not be available under windows environment
             pass
diff --git a/py/selenium/webdriver/firefox/firefox_binary.py b/py/selenium/webdriver/firefox/firefox_binary.py
@@ -110,7 +110,7 @@ def _find_exe_in_registry(self):
                 key = OpenKey(HKEY_LOCAL_MACHINE, path)
                 command = QueryValue(key, "")
                 break
-            except WindowsError:
+            except OSError:
                 pass
         else:
             return ""

diff --git a/py/selenium/webdriver/phantomjs/service.py b/py/selenium/webdriver/phantomjs/service.py
@@ -88,6 +88,6 @@ def stop(self):
             if self.process:
                 self.process.kill()
                 self.process.wait()
-        except WindowsError:
+        except OSError:
             # kill may not be available under windows environment
             pass