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

Unified Diff: chrome/browser/ui/views/frame/contents_container.h

Issue 5997004: Merge 69201 - Fixes bug in instant that resulted in flickery fade. The proble... (Closed) Base URL: svn://svn.chromium.org/chrome/branches/597/src/
Patch Set: '' Created 10 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/views/frame/browser_view.cc ('k') | chrome/browser/ui/views/frame/contents_container.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/frame/contents_container.h
===================================================================
--- chrome/browser/ui/views/frame/contents_container.h (revision 69571)
+++ chrome/browser/ui/views/frame/contents_container.h (working copy)
@@ -45,6 +45,9 @@
// Fades out the active contents.
void FadeActiveContents();
+ // Shows the fade. This is similiar to |FadeActiveContents|, but is immediate.
+ void ShowFade();
+
// Removes the fade. This is done implicitly when the preview is made active.
void RemoveFade();
@@ -55,6 +58,14 @@
virtual void AnimationProgressed(const Animation* animation);
private:
+ class OverlayContentView;
+
+ // Creates the overlay widget. The opacity is set at |initial_opacity|.
+ void CreateOverlay(int initial_opacity);
+
+ // Invoked when the contents view of the overlay is destroyed.
+ void OverlayViewDestroyed();
+
views::View* active_;
views::View* preview_;
@@ -65,6 +76,9 @@
// make the active view appear faded out.
views::Widget* active_overlay_;
+ // Content view of active_overlay. Used to track when the widget is destroyed.
+ OverlayContentView* overlay_view_;
+
// Animation used to vary the opacity of active_overlay.
scoped_ptr<SlideAnimation> overlay_animation_;
« no previous file with comments | « chrome/browser/ui/views/frame/browser_view.cc ('k') | chrome/browser/ui/views/frame/contents_container.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698