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

Unified Diff: chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellTab.java

Issue 481803004: Removing ContentViewCore dependencies from few functions which acts as direct wrapper to WebContents (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed the nit. Created 6 years, 4 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: chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellTab.java
diff --git a/chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellTab.java b/chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellTab.java
index ba7e758e83793717c0a5114caccf75a3a007a485..9b27280db1a31d3c10985a65478da98b7c201cbe 100644
--- a/chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellTab.java
+++ b/chrome/android/shell/java/src/org/chromium/chrome/shell/ChromeShellTab.java
@@ -63,7 +63,7 @@ public class ChromeShellTab extends Tab {
if (TextUtils.isEmpty(url)) return;
ContentViewCore contentViewCore = getContentViewCore();
Yaron 2014/08/18 23:16:24 We should remove the CVC dependency entirely when
AKVT 2014/09/11 17:19:43 Done. Thanks
- if (TextUtils.equals(url, contentViewCore.getUrl())) {
+ if (TextUtils.equals(url, contentViewCore.getWebContents().getUrl())) {
contentViewCore.reload(true);
} else {
if (postData == null) {

Powered by Google App Engine
This is Rietveld 408576698