Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(531)

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

Issue 10969045: Revert 158067 - Remove native side of content_view_client (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/public/android/java/src/org/chromium/content/browser/accessibility/AccessibilityInjector.java
===================================================================
--- content/public/android/java/src/org/chromium/content/browser/accessibility/AccessibilityInjector.java (revision 158071)
+++ content/public/android/java/src/org/chromium/content/browser/accessibility/AccessibilityInjector.java (working copy)
@@ -19,7 +19,6 @@
import org.apache.http.client.utils.URLEncodedUtils;
import org.chromium.content.browser.ContentViewCore;
import org.chromium.content.browser.JavascriptInterface;
-import org.chromium.content.browser.WebContentsObserverAndroid;
import org.json.JSONException;
import org.json.JSONObject;
@@ -34,7 +33,7 @@
/**
* Responsible for accessibility injection and management of a {@link ContentViewCore}.
*/
-public class AccessibilityInjector extends WebContentsObserverAndroid {
+public class AccessibilityInjector {
// The ContentView this injector is responsible for managing.
protected ContentViewCore mContentViewCore;
@@ -94,7 +93,6 @@
* @param view The ContentViewCore that this AccessibilityInjector manages.
*/
protected AccessibilityInjector(ContentViewCore view) {
- super(view);
mContentViewCore = view;
}
@@ -187,16 +185,10 @@
* accessibility script as not being injected. This way we can properly ignore incoming
* accessibility gesture events.
*/
- @Override
- public void didStartLoading(String url) {
+ public void onPageLoadStarted() {
mScriptInjected = false;
}
- @Override
- public void didStopLoading(String url) {
- injectAccessibilityScriptIntoPage();
- }
-
/**
* Stop any notifications that are currently going on (e.g. Text-to-Speech).
*/
Property changes on: content/public/android/java/src/org/chromium/content/browser/accessibility/AccessibilityInjector.java
___________________________________________________________________
Added: svn:mergeinfo

Powered by Google App Engine
This is Rietveld 408576698