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

Side by Side Diff: chrome/browser/tab_contents/background_contents.h

Issue 9003014: Replace WebUI::tab_contents() with web_contents() and switch all users to use web_contents.h inst... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 8 years, 11 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/browser/profiles/profile.cc ('k') | chrome/browser/tab_contents/background_contents.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_TAB_CONTENTS_BACKGROUND_CONTENTS_H_ 5 #ifndef CHROME_BROWSER_TAB_CONTENTS_BACKGROUND_CONTENTS_H_
6 #define CHROME_BROWSER_TAB_CONTENTS_BACKGROUND_CONTENTS_H_ 6 #define CHROME_BROWSER_TAB_CONTENTS_BACKGROUND_CONTENTS_H_
7 #pragma once 7 #pragma once
8 8
9 #include <string> 9 #include <string>
10 10
(...skipping 26 matching lines...) Expand all
37 protected: 37 protected:
38 virtual ~Delegate() {} 38 virtual ~Delegate() {}
39 }; 39 };
40 40
41 BackgroundContents(SiteInstance* site_instance, 41 BackgroundContents(SiteInstance* site_instance,
42 int routing_id, 42 int routing_id,
43 Delegate* delegate); 43 Delegate* delegate);
44 virtual ~BackgroundContents(); 44 virtual ~BackgroundContents();
45 45
46 TabContents* tab_contents() { return tab_contents_.get(); } 46 TabContents* tab_contents() { return tab_contents_.get(); }
47 // TODO(jam): move impl to header once this class only uses WebContents.
48 content::WebContents* web_contents() const;
47 virtual const GURL& GetURL() const; 49 virtual const GURL& GetURL() const;
48 50
49 // content::WebContentsDelegate implementation: 51 // content::WebContentsDelegate implementation:
50 virtual void CloseContents(content::WebContents* source) OVERRIDE; 52 virtual void CloseContents(content::WebContents* source) OVERRIDE;
51 virtual bool ShouldSuppressDialogs() OVERRIDE; 53 virtual bool ShouldSuppressDialogs() OVERRIDE;
52 virtual void DidNavigateMainFramePostCommit( 54 virtual void DidNavigateMainFramePostCommit(
53 content::WebContents* tab) OVERRIDE; 55 content::WebContents* tab) OVERRIDE;
54 virtual void AddNewContents(content::WebContents* source, 56 virtual void AddNewContents(content::WebContents* source,
55 content::WebContents* new_contents, 57 content::WebContents* new_contents,
56 WindowOpenDisposition disposition, 58 WindowOpenDisposition disposition,
(...skipping 29 matching lines...) Expand all
86 BackgroundContents* contents; 88 BackgroundContents* contents;
87 89
88 // The name of the parent frame for these contents. 90 // The name of the parent frame for these contents.
89 const string16& frame_name; 91 const string16& frame_name;
90 92
91 // The ID of the parent application (if any). 93 // The ID of the parent application (if any).
92 const string16& application_id; 94 const string16& application_id;
93 }; 95 };
94 96
95 #endif // CHROME_BROWSER_TAB_CONTENTS_BACKGROUND_CONTENTS_H_ 97 #endif // CHROME_BROWSER_TAB_CONTENTS_BACKGROUND_CONTENTS_H_
OLDNEW
« no previous file with comments | « chrome/browser/profiles/profile.cc ('k') | chrome/browser/tab_contents/background_contents.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698