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

Unified Diff: content/browser/renderer_host/software_frame_manager.h

Issue 43193002: Aura/ÜC: Drop frames on background tabs (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: CONTENT_EXPORT Created 7 years, 2 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: content/browser/renderer_host/software_frame_manager.h
diff --git a/content/browser/renderer_host/software_frame_manager.h b/content/browser/renderer_host/software_frame_manager.h
index d7f186f41ff8457496f77aef32c55f18cf0b88cf..503a40d20e73a41efb0a33c89a196045d812650f 100644
--- a/content/browser/renderer_host/software_frame_manager.h
+++ b/content/browser/renderer_host/software_frame_manager.h
@@ -11,17 +11,16 @@
#include "base/basictypes.h"
#include "base/memory/ref_counted.h"
#include "base/memory/shared_memory.h"
-#include "base/memory/singleton.h"
#include "base/memory/weak_ptr.h"
#include "cc/output/software_frame_data.h"
#include "cc/resources/single_release_callback.h"
#include "cc/resources/texture_mailbox.h"
+#include "content/browser/renderer_host/renderer_frame_manager.h"
#include "content/common/content_export.h"
#include "ui/gfx/size.h"
namespace content {
class SoftwareFrame;
-class SoftwareFrameMemoryManager;
class CONTENT_EXPORT SoftwareFrameManagerClient {
public:
@@ -36,11 +35,11 @@ class CONTENT_EXPORT SoftwareFrameManagerClient {
virtual void ReleaseReferencesToSoftwareFrame() = 0;
};
-class CONTENT_EXPORT SoftwareFrameManager {
+class CONTENT_EXPORT SoftwareFrameManager : public RendererFrameManagerClient {
public:
explicit SoftwareFrameManager(
base::WeakPtr<SoftwareFrameManagerClient> client);
- ~SoftwareFrameManager();
+ virtual ~SoftwareFrameManager();
// Swaps to a new frame from shared memory. This frame is guaranteed to
// not be evicted until SwapToNewFrameComplete is called.
@@ -61,11 +60,9 @@ class CONTENT_EXPORT SoftwareFrameManager {
gfx::Size GetCurrentFrameSizeInDIP() const;
private:
- friend class SoftwareFrameMemoryManager;
-
// Called by SoftwareFrameMemoryManager to demand that the current frame
// be evicted.
- void EvictCurrentFrame();
+ virtual void EvictCurrentFrame() OVERRIDE;
base::WeakPtr<SoftwareFrameManagerClient> client_;
@@ -75,31 +72,6 @@ class CONTENT_EXPORT SoftwareFrameManager {
DISALLOW_COPY_AND_ASSIGN(SoftwareFrameManager);
};
-class CONTENT_EXPORT SoftwareFrameMemoryManager {
- public:
- static SoftwareFrameMemoryManager* GetInstance();
-
- void AddFrame(SoftwareFrameManager*, bool visible);
- void RemoveFrame(SoftwareFrameManager*);
- void SetFrameVisibility(SoftwareFrameManager*, bool visible);
-
- size_t max_number_of_saved_frames() const {
- return max_number_of_saved_frames_;
- }
-
- private:
- SoftwareFrameMemoryManager();
- ~SoftwareFrameMemoryManager();
- void CullHiddenFrames();
- friend struct DefaultSingletonTraits<SoftwareFrameMemoryManager>;
-
- std::set<SoftwareFrameManager*> visible_frames_;
- std::list<SoftwareFrameManager*> hidden_frames_;
- size_t max_number_of_saved_frames_;
-
- DISALLOW_COPY_AND_ASSIGN(SoftwareFrameMemoryManager);
-};
-
} // namespace content
#endif // CONTENT_BROWSER_RENDERER_HOST_SOFTWARE_FRAME_MANAGER_H_
« no previous file with comments | « content/browser/renderer_host/renderer_frame_manager.cc ('k') | content/browser/renderer_host/software_frame_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698