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

Unified Diff: blimp/client/public/contents/blimp_contents.h

Issue 2058263002: Tied up BlimpNavigationController to NavigationFeature (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@blimp_core
Patch Set: Removed BlimpContentsTest.java 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 side-by-side diff with in-line comments
Download patch
Index: blimp/client/public/contents/blimp_contents.h
diff --git a/blimp/client/public/contents/blimp_contents.h b/blimp/client/public/contents/blimp_contents.h
index 02d7560211c2abf6b8ef196413544039c8a9ad88..ea11aa7305a00239917de041d3f01b50c2fd34d8 100644
--- a/blimp/client/public/contents/blimp_contents.h
+++ b/blimp/client/public/contents/blimp_contents.h
@@ -15,10 +15,12 @@ namespace client {
class BlimpContentsObserver;
class BlimpNavigationController;
-// BlimpContents is the core class in //blimp/client/core. It renders web pages
-// from an engine in a rectangular area.
-// It enables callers to control the blimp engine through the use of the
-// navigation controller.
+// BlimpContents is the core class in blimp client which is responsible for
+// rendering web pages in a rectangular area. The interaction with between the
+// client and engine is encapsulated behind this interface through the use of
+// the navigation controller. Each BlimpContents has exactly one
+// BlimpNavigationController which can be used to load URLs, navigate
+// backwards/forward etc. See blimp_navigation_controller.h for more details.
class BlimpContents : public base::SupportsUserData {
public:
// Retrives the navigation controller that controls all navigation related

Powered by Google App Engine
This is Rietveld 408576698