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

Unified Diff: content/browser/host_zoom_level_context.h

Issue 393133002: Migrate HostZoomMap to live in StoragePartition. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add proper method mock for BrowserContext::CreateZoomLevelDelegate() override. Created 6 years, 1 month 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/host_zoom_level_context.h
diff --git a/content/browser/host_zoom_level_context.h b/content/browser/host_zoom_level_context.h
new file mode 100644
index 0000000000000000000000000000000000000000..01e267a9d0b2afd8b8e935c4286c285510bbc277
--- /dev/null
+++ b/content/browser/host_zoom_level_context.h
@@ -0,0 +1,65 @@
+// Copyright (c) 2014 The Chromium Authors. All rights reserved.
Charlie Reis 2014/11/11 05:29:22 nit: No (c)
wjmaclean 2014/11/11 19:20:52 Done. (I thought I had cleaned all those up ... oh
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CONTENT_BROWSER_HOST_ZOOM_LEVEL_CONTEXT_H_
+#define CONTENT_BROWSER_HOST_ZOOM_LEVEL_CONTEXT_H_
+
+#include "base/macros.h"
+#include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
+#include "content/browser/host_zoom_map_impl.h"
+#include "content/public/browser/zoom_level_delegate.h"
+
+class PrefService;
+
+namespace base {
+class FilePath;
Charlie Reis 2014/11/11 05:29:22 Looks unnecessary.
wjmaclean 2014/11/11 19:20:52 Done. (Similarly, thought I had removed this too .
+}
+
+namespace content {
+struct HostZoomLevelContextDeleter;
+
+// This class manages a HostZoomMap and associates it with a ZoomLevelDelegate,
+// if one is provided. It also serves to keep the zoom level machinery details
+// separate from the owning StoragePartitionImpl.
+class HostZoomLevelContext
+ : public base::RefCountedThreadSafe<HostZoomLevelContext,
+ HostZoomLevelContextDeleter> {
+ public:
+ explicit HostZoomLevelContext(
+ scoped_ptr<ZoomLevelDelegate> zoom_level_delegate);
+
+ HostZoomMap* GetHostZoomMap() const { return host_zoom_map_impl_.get(); }
+ ZoomLevelDelegate* GetZoomLevelDelegate() const {
+ return zoom_level_delegate_.get();
+ }
+
+ protected:
+ virtual ~HostZoomLevelContext();
+
+ private:
+ friend class base::DeleteHelper<HostZoomLevelContext>;
+ friend class base::RefCountedThreadSafe<HostZoomLevelContext,
+ HostZoomLevelContextDeleter>;
+ friend struct HostZoomLevelContextDeleter;
+
+ void DeleteOnCorrectThread() const;
+
+ scoped_ptr<HostZoomMapImpl> host_zoom_map_impl_;
+ // Release the delegate before the HostZoomMap, in case it is carrying
+ // any HostZoomMap::Subscription pointers.
+ scoped_ptr<ZoomLevelDelegate> zoom_level_delegate_;
+
+ DISALLOW_COPY_AND_ASSIGN(HostZoomLevelContext);
+};
+
+struct HostZoomLevelContextDeleter {
+ static void Destruct(const HostZoomLevelContext* context) {
+ context->DeleteOnCorrectThread();
+ }
+};
+
+} // namespace content
+
+#endif // CONTENT_BROWSER_HOST_ZOOM_LEVEL_CONTEXT_H_

Powered by Google App Engine
This is Rietveld 408576698