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

Unified Diff: chrome/browser/printing/background_printing_manager.h

Issue 10535045: TabContentsWrapper -> TabContents, for printing code. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 years, 6 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
« no previous file with comments | « no previous file | chrome/browser/printing/background_printing_manager.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/printing/background_printing_manager.h
===================================================================
--- chrome/browser/printing/background_printing_manager.h (revision 140896)
+++ chrome/browser/printing/background_printing_manager.h (working copy)
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
@@ -15,7 +15,6 @@
#include "content/public/browser/notification_registrar.h"
class TabContents;
-typedef TabContents TabContentsWrapper;
namespace content {
class RenderProcessHost;
@@ -29,22 +28,22 @@
class BackgroundPrintingManager : public base::NonThreadSafe,
public content::NotificationObserver {
public:
- typedef std::set<TabContentsWrapper*> TabContentsWrapperSet;
+ typedef std::set<TabContents*> TabContentsSet;
BackgroundPrintingManager();
virtual ~BackgroundPrintingManager();
// Takes ownership of |preview_tab| and deletes it when |preview_tab| finishes
- // printing. This removes the TabContentsWrapper from its TabStrip and
- // hides it from the user.
- void OwnPrintPreviewTab(TabContentsWrapper* preview_tab);
+ // printing. This removes the TabContents from its TabStrip and hides it from
+ // the user.
+ void OwnPrintPreviewTab(TabContents* preview_tab);
// Let others iterate over the list of background printing tabs.
- TabContentsWrapperSet::const_iterator begin();
- TabContentsWrapperSet::const_iterator end();
+ TabContentsSet::const_iterator begin();
+ TabContentsSet::const_iterator end();
// Returns true if |printing_tabs_| contains |preview_tab|.
- bool HasPrintPreviewTab(TabContentsWrapper* preview_tab);
+ bool HasPrintPreviewTab(TabContents* preview_tab);
// content::NotificationObserver overrides:
virtual void Observe(int type,
@@ -54,23 +53,23 @@
private:
// Notifications handlers.
void OnRendererProcessClosed(content::RenderProcessHost* rph);
- void OnPrintJobReleased(TabContentsWrapper* preview_tab);
- void OnTabContentsDestroyed(TabContentsWrapper* preview_tab);
+ void OnPrintJobReleased(TabContents* preview_tab);
+ void OnTabContentsDestroyed(TabContents* preview_tab);
// Add |tab| to the pending deletion set and schedule deletion.
- void DeletePreviewTab(TabContentsWrapper* tab);
+ void DeletePreviewTab(TabContents* tab);
- // Check if any of the TabContentsWrappers in |set| share a RenderProcessHost
+ // Check if any of the TabContents in |set| share a RenderProcessHost
// with |tab|, excluding |tab|.
- bool HasSharedRenderProcessHost(const TabContentsWrapperSet& set,
- TabContentsWrapper* tab);
+ bool HasSharedRenderProcessHost(const TabContentsSet& set,
+ TabContents* tab);
// The set of print preview tabs managed by BackgroundPrintingManager.
- TabContentsWrapperSet printing_tabs_;
+ TabContentsSet printing_tabs_;
// The set of print preview tabs managed by BackgroundPrintingManager that
// are pending deletion.
- TabContentsWrapperSet printing_tabs_pending_deletion_;
+ TabContentsSet printing_tabs_pending_deletion_;
content::NotificationRegistrar registrar_;
« no previous file with comments | « no previous file | chrome/browser/printing/background_printing_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698