diff options
| author | grt <grt@chromium.org> | 2015-02-26 06:26:53 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-02-26 14:27:45 +0000 |
| commit | c2d96fa87b653355fc8335c0accecff8852cae5d (patch) | |
| tree | f4c0a61e47ed90ba37f2b2b9ba4d78dad1d80677 /android_webview/native | |
| parent | 49001892c02a5287233acc68edaaf2e0dc981163 (diff) | |
| download | chromium_src-c2d96fa87b653355fc8335c0accecff8852cae5d.zip chromium_src-c2d96fa87b653355fc8335c0accecff8852cae5d.tar.gz chromium_src-c2d96fa87b653355fc8335c0accecff8852cae5d.tar.bz2 | |
Revert of [Android WebView] Synthesize a fake page loading event on page source modification (patchset #8 id:140001 of https://codereview.chromium.org/924833003/)
Reason for revert:
Reverting to see if this caused AwContentsClientFullScreenTest#testPowerSaveBlockerIsTransferredToFullscreen failure here: http://crbug.com/462213.
Original issue's description:
> [Android WebView] Synthesize a fake page loading event on page source modification
>
> When a script modifies page source of a non-committed page, we need to
> notify clients, so they can update the URL bar to avoid confusion.
>
> BUG=458569
>
> Committed: https://crrev.com/e38bb6cdeb1170cb2cd7faae8507dc6c34c58c7d
> Cr-Commit-Position: refs/heads/master@{#318222}
TBR=torne@chromium.org,tedchoc@chromium.org,davidben@chromium.org,creis@chromium.org,mnaganov@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=458569
Review URL: https://codereview.chromium.org/958933002
Cr-Commit-Position: refs/heads/master@{#318231}
Diffstat (limited to 'android_webview/native')
| -rw-r--r-- | android_webview/native/aw_web_contents_delegate.cc | 12 | ||||
| -rw-r--r-- | android_webview/native/aw_web_contents_delegate.h | 2 |
2 files changed, 0 insertions, 14 deletions
diff --git a/android_webview/native/aw_web_contents_delegate.cc b/android_webview/native/aw_web_contents_delegate.cc index d2a71e6..acece1b 100644 --- a/android_webview/native/aw_web_contents_delegate.cc +++ b/android_webview/native/aw_web_contents_delegate.cc @@ -163,18 +163,6 @@ void AwWebContentsDelegate::AddNewContents(WebContents* source, } } -void AwWebContentsDelegate::NavigationStateChanged( - content::WebContents* source, - content::InvalidateTypes changed_flags) { - JNIEnv* env = AttachCurrentThread(); - - ScopedJavaLocalRef<jobject> java_delegate = GetJavaDelegate(env); - if (java_delegate.obj()) { - Java_AwWebContentsDelegate_navigationStateChanged(env, java_delegate.obj(), - changed_flags); - } -} - // Notifies the delegate about the creation of a new WebContents. This // typically happens when popups are created. void AwWebContentsDelegate::WebContentsCreated( diff --git a/android_webview/native/aw_web_contents_delegate.h b/android_webview/native/aw_web_contents_delegate.h index afc881b..96cd325 100644 --- a/android_webview/native/aw_web_contents_delegate.h +++ b/android_webview/native/aw_web_contents_delegate.h @@ -40,8 +40,6 @@ class AwWebContentsDelegate bool user_gesture, bool* was_blocked) override; - void NavigationStateChanged(content::WebContents* source, - content::InvalidateTypes changed_flags) override; void WebContentsCreated(content::WebContents* source_contents, int opener_render_frame_id, const base::string16& frame_name, |