Removing ContentViewCore dependencies from few functions which acts as direct wrapper to WebContents

content/public/android/java/src/org/chromium/content/browser/accessibility/AccessibilityInjector.java

Index: content/public/android/java/src/org/chromium/content/browser/accessibility/AccessibilityInjector.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/accessibility/AccessibilityInjector.java b/content/public/android/java/src/org/chromium/content/browser/accessibility/AccessibilityInjector.java
index de4db4aff4afd4b8b61b03c3fc27e37f40635c6d..e5dec6e1229467c84685c349b294258c785871e2 100644
--- a/content/public/android/java/src/org/chromium/content/browser/accessibility/AccessibilityInjector.java
+++ b/content/public/android/java/src/org/chromium/content/browser/accessibility/AccessibilityInjector.java
@@ -305,11 +305,13 @@ public class AccessibilityInjector extends WebContentsObserverAndroid {
     }
 
     private int getAxsUrlParameterValue() {
-        if (mContentViewCore.getUrl() == null) return ACCESSIBILITY_SCRIPT_INJECTION_UNDEFINED;
+        if (mContentViewCore.getWebContents().getUrl() == null) {

[Yaron, 2014/08/18 23:16:24]

So this is probably a few steps down the road (i.e. a few blocking issues in the
way) but I'm curious whether we'd be able to have some of these classes only
depend on WebContents and lower layers. What would be preventing us from doing
that? It seems like we need the Context and the ContainerView, the latter is
supposed to be available from webcontents based on web_contents.h
(GetNativeView). 

[AKVT, 2014/09/11 17:19:43]

@Yaron, Can we do this with a follow up patch ? as it needs more changes

[AKVT, 2014/10/13 12:25:05]

@Yaron, I would like to take a follow up to this CL.
Please help me to address following issues.

How we can place addJavascriptInterface() ? Can we move the native part
including java_bridge_dispatcher_host_ to WebContents ?

ContentSettings can we manage inside WebContents instead of CVC ?

+            return ACCESSIBILITY_SCRIPT_INJECTION_UNDEFINED;
+        }
 
         try {
-            List<NameValuePair> params = URLEncodedUtils.parse(new URI(mContentViewCore.getUrl()),
-                    null);
+            List<NameValuePair> params = URLEncodedUtils.parse(
+                    new URI(mContentViewCore.getWebContents().getUrl()), null);
 
             for (NameValuePair param : params) {
                 if ("axs".equals(param.getName())) {
@@ -317,8 +319,11 @@ public class AccessibilityInjector extends WebContentsObserverAndroid {
                 }
             }
         } catch (URISyntaxException ex) {
+            // Intentional no-op.
         } catch (NumberFormatException ex) {
+            // Intentional no-op.
         } catch (IllegalArgumentException ex) {
+            // Intentional no-op.
         }
 
         return ACCESSIBILITY_SCRIPT_INJECTION_UNDEFINED;