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

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

Issue 2233023002: Adding BlimpNavigationController to Tab (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@nav_handler_remove
Patch Set: @nyquist feedback Created 4 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 package org.chromium.chrome.browser.tab;
6
7 import org.chromium.base.ObserverList.RewindableIterator;
8 import org.chromium.blimp_public.contents.BlimpContentsObserver;
9
10 /**
11 * BlimpContentsObserver used by Tab.
12 */
13 public class TabBlimpContentsObserver implements BlimpContentsObserver {
14 private Tab mTab;
15
16 public TabBlimpContentsObserver(Tab tab) {
17 mTab = tab;
18 }
19
20 @Override
21 public void onNavigationStateChanged() {
David Trainor- moved to gerrit 2016/08/18 17:06:30 Can you put a comment that we want to notify all U
shaktisahu 2016/08/18 19:36:06 Done.
22 mTab.updateTitle();
23 RewindableIterator<TabObserver> observers = mTab.getTabObservers();
24 while (observers.hasNext()) {
25 observers.next().onUrlUpdated(mTab);
26 }
27 }
28 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698