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

Side by Side Diff: chrome/android/java/src/org/chromium/chrome/browser/tab/TabWebContentsObserver.java

Issue 2380743003: Refactor ContentViewClient (2/6) (Closed)
Patch Set: // Do nothing Created 4 years 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.chrome.browser.tab; 5 package org.chromium.chrome.browser.tab;
6 6
7 import android.os.SystemClock; 7 import android.os.SystemClock;
8 import android.support.annotation.IntDef; 8 import android.support.annotation.IntDef;
9 import android.view.View; 9 import android.view.View;
10 10
(...skipping 166 matching lines...) Expand 10 before | Expand all | Expand 10 after
177 ((ChromeApplication) mTab.getApplicationContext()).getPolicyAudi tor(); 177 ((ChromeApplication) mTab.getApplicationContext()).getPolicyAudi tor();
178 auditor.notifyAuditEvent(mTab.getApplicationContext(), AuditEvent.OPEN_U RL_FAILURE, 178 auditor.notifyAuditEvent(mTab.getApplicationContext(), AuditEvent.OPEN_U RL_FAILURE,
179 failingUrl, description); 179 failingUrl, description);
180 if (errorCode == BLOCKED_BY_ADMINISTRATOR) { 180 if (errorCode == BLOCKED_BY_ADMINISTRATOR) {
181 auditor.notifyAuditEvent( 181 auditor.notifyAuditEvent(
182 mTab.getApplicationContext(), AuditEvent.OPEN_URL_BLOCKED, f ailingUrl, ""); 182 mTab.getApplicationContext(), AuditEvent.OPEN_URL_BLOCKED, f ailingUrl, "");
183 } 183 }
184 } 184 }
185 185
186 @Override 186 @Override
187 public void titleWasSet(String title) {
188 mTab.updateTitle(title);
189 }
190
191 @Override
187 public void didStartProvisionalLoadForFrame(long frameId, long parentFrameId , 192 public void didStartProvisionalLoadForFrame(long frameId, long parentFrameId ,
188 boolean isMainFrame, String validatedUrl, boolean isErrorPage, 193 boolean isMainFrame, String validatedUrl, boolean isErrorPage,
189 boolean isIframeSrcdoc) { 194 boolean isIframeSrcdoc) {
190 if (isMainFrame) mTab.didStartPageLoad(validatedUrl, isErrorPage); 195 if (isMainFrame) mTab.didStartPageLoad(validatedUrl, isErrorPage);
191 196
192 mTab.handleDidStartProvisionalLoadForFrame(isMainFrame, validatedUrl); 197 mTab.handleDidStartProvisionalLoadForFrame(isMainFrame, validatedUrl);
193 } 198 }
194 199
195 @Override 200 @Override
196 public void didCommitProvisionalLoadForFrame(long frameId, boolean isMainFra me, String url, 201 public void didCommitProvisionalLoadForFrame(long frameId, boolean isMainFra me, String url,
(...skipping 111 matching lines...) Expand 10 before | Expand all | Expand 10 after
308 } 313 }
309 } 314 }
310 315
311 @Override 316 @Override
312 public void destroy() { 317 public void destroy() {
313 MediaCaptureNotificationService.updateMediaNotificationForTab( 318 MediaCaptureNotificationService.updateMediaNotificationForTab(
314 mTab.getApplicationContext(), mTab.getId(), 0, mTab.getUrl()); 319 mTab.getApplicationContext(), mTab.getId(), 0, mTab.getUrl());
315 super.destroy(); 320 super.destroy();
316 } 321 }
317 } 322 }
OLDNEW
« no previous file with comments | « chrome/android/java/src/org/chromium/chrome/browser/tab/Tab.java ('k') | content/browser/android/content_view_core_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698