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

Side by Side Diff: blimp/client/core/contents/blimp_navigation_controller_delegate.h

Issue 2320923002: Add a full Blimp integration test. (Closed)
Patch Set: Fix build break with chrome embedder Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 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 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 #ifndef BLIMP_CLIENT_CORE_CONTENTS_BLIMP_NAVIGATION_CONTROLLER_DELEGATE_H_ 5 #ifndef BLIMP_CLIENT_CORE_CONTENTS_BLIMP_NAVIGATION_CONTROLLER_DELEGATE_H_
6 #define BLIMP_CLIENT_CORE_CONTENTS_BLIMP_NAVIGATION_CONTROLLER_DELEGATE_H_ 6 #define BLIMP_CLIENT_CORE_CONTENTS_BLIMP_NAVIGATION_CONTROLLER_DELEGATE_H_
7 7
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "url/gurl.h" 9 #include "url/gurl.h"
10 10
11 namespace blimp { 11 namespace blimp {
12 namespace client { 12 namespace client {
13 13
14 // Interface for objects embedding a BlimpNavigationControllerImpl to provide 14 // Interface for objects embedding a BlimpNavigationControllerImpl to provide
15 // the functionality BlimpNavigationControllerImpl needs. 15 // the functionality BlimpNavigationControllerImpl needs.
16 class BlimpNavigationControllerDelegate { 16 class BlimpNavigationControllerDelegate {
17 public: 17 public:
18 virtual ~BlimpNavigationControllerDelegate() = default; 18 virtual ~BlimpNavigationControllerDelegate() = default;
19 19
20 // TODO(dtrainor): Pull apart this method into more fine grained notifications 20 // TODO(dtrainor): Pull apart this method into more fine grained notifications
21 // or add an enum to detect exactly which state changed. 21 // or add an enum to detect exactly which state changed.
22 // Informs the delegate that navigation state has changed. 22 // Informs the delegate that navigation state has changed.
23 virtual void OnNavigationStateChanged() = 0; 23 virtual void OnNavigationStateChanged() = 0;
24 24
25 // Informs the delegate that navigation loading has started or stopped. 25 // Informs the delegate that navigation loading has started or stopped.
26 virtual void OnLoadingStateChanged(bool loading) = 0; 26 virtual void OnLoadingStateChanged(bool loading) = 0;
27 27
28 // Informs the delegate that page loading has started or stopped.
29 virtual void OnPageLoadingStateChanged(bool loading) = 0;
30
28 protected: 31 protected:
29 BlimpNavigationControllerDelegate() {} 32 BlimpNavigationControllerDelegate() {}
30 33
31 private: 34 private:
32 DISALLOW_COPY_AND_ASSIGN(BlimpNavigationControllerDelegate); 35 DISALLOW_COPY_AND_ASSIGN(BlimpNavigationControllerDelegate);
33 }; 36 };
34 37
35 } // namespace client 38 } // namespace client
36 } // namespace blimp 39 } // namespace blimp
37 40
38 #endif // BLIMP_CLIENT_CORE_CONTENTS_BLIMP_NAVIGATION_CONTROLLER_DELEGATE_H_ 41 #endif // BLIMP_CLIENT_CORE_CONTENTS_BLIMP_NAVIGATION_CONTROLLER_DELEGATE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698