...some cleaning

appium · Jul 6, 2015 · 3628627 · 3628627
1 parent 9d1a1b5
commit 3628627
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/src/main/java/io/appium/java_client/pagefactory/AppiumElementLocator.java b/src/main/java/io/appium/java_client/pagefactory/AppiumElementLocator.java
@@ -106,8 +106,9 @@ private String getPlatform(){
         if (IOSDriver.class.isAssignableFrom(driverClass))
             return MobilePlatform.IOS;
 
-        //it is possible that here customers implementation of WebDriver
-        //At this case it has to implement HasCapabilities
+        //it is possible that somebody uses RemoteWebDriver or their
+        //own WebDriver implementation. At this case capabilities are used
+        //to detect platform
         if (HasCapabilities.class.isAssignableFrom(driverClass))
             return String.valueOf(((HasCapabilities) d).getCapabilities().
                     getCapability(MobileCapabilityType.PLATFORM_NAME));

diff --git a/src/main/java/io/appium/java_client/pagefactory/ContentMappedBy.java b/src/main/java/io/appium/java_client/pagefactory/ContentMappedBy.java
@@ -23,8 +23,6 @@ private By returnRelevantBy(SearchContext context){
 			return map.get(ContentType.HTML);
 		}
 
-        By result = null;
-
 		ContextAware contextAware = ContextAware.class.cast(driver);
 		String currentContext = contextAware.getContext();
 		if (currentContext.contains(NATIVE_APP_PATTERN))