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

Unified Diff: athena/content/web_activity.h

Issue 536013002: Decoupling visibility states from webcontent, adding visibility management in ResourceManager (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added a few more unittests Created 6 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 side-by-side diff with in-line comments
Download patch
Index: athena/content/web_activity.h
diff --git a/athena/content/web_activity.h b/athena/content/web_activity.h
index e421f6f621e66dc186e5e19f192cebd27dd0dfae..a918bcd3f817e93cf1b588119ea7a0fae93d79f1 100644
--- a/athena/content/web_activity.h
+++ b/athena/content/web_activity.h
@@ -30,10 +30,11 @@ class WebActivity : public Activity,
public:
WebActivity(content::BrowserContext* context, const GURL& gurl);
WebActivity(AthenaWebView* web_view);
- virtual ~WebActivity();
protected:
- // Activity:
+ virtual ~WebActivity();
+
+ // Activity:
virtual athena::ActivityViewModel* GetActivityViewModel() OVERRIDE;
virtual void SetCurrentState(ActivityState state) OVERRIDE;
virtual ActivityState GetCurrentState() OVERRIDE;
@@ -58,6 +59,17 @@ class WebActivity : public Activity,
virtual void DidChangeThemeColor(SkColor theme_color) OVERRIDE;
private:
+ // Make the content visible. This call should only be paired with
+ // MakeInvisible. Note: Upon object creation the content is visible.
+ void MakeVisible();
+
+ // Make the content invisible. This call should only be paired with
+ // MakeVisible.
+ void MakeInvisible();
+
+ // Reload the content if required, and start observing it.
+ void ReloadAndObserve();
+
content::BrowserContext* browser_context_;
const GURL url_;
AthenaWebView* web_view_;

Powered by Google App Engine
This is Rietveld 408576698