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

Side by Side Diff: chrome/browser/search_engines/search_provider_install_state_message_filter.h

Issue 10071036: RefCounted types should not have public destructors, chrome/browser/ part 6 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Implementation fixes Created 8 years, 8 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
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_SEARCH_ENGINES_SEARCH_PROVIDER_INSTALL_STATE_MESSAGE_FILT ER_H_ 5 #ifndef CHROME_BROWSER_SEARCH_ENGINES_SEARCH_PROVIDER_INSTALL_STATE_MESSAGE_FILT ER_H_
6 #define CHROME_BROWSER_SEARCH_ENGINES_SEARCH_PROVIDER_INSTALL_STATE_MESSAGE_FILT ER_H_ 6 #define CHROME_BROWSER_SEARCH_ENGINES_SEARCH_PROVIDER_INSTALL_STATE_MESSAGE_FILT ER_H_
7 7
8 #include "base/memory/weak_ptr.h" 8 #include "base/memory/weak_ptr.h"
9 #include "chrome/browser/search_engines/search_provider_install_data.h" 9 #include "chrome/browser/search_engines/search_provider_install_data.h"
10 #include "chrome/common/search_provider.h" 10 #include "chrome/common/search_provider.h"
11 #include "content/public/browser/browser_message_filter.h" 11 #include "content/public/browser/browser_message_filter.h"
12 12
13 class GURL; 13 class GURL;
14 class Profile; 14 class Profile;
15 15
16 // Handles messages regarding search provider install state on the I/O thread. 16 // Handles messages regarding search provider install state on the I/O thread.
17 class SearchProviderInstallStateMessageFilter 17 class SearchProviderInstallStateMessageFilter
18 : public content::BrowserMessageFilter { 18 : public content::BrowserMessageFilter {
19 public: 19 public:
20 // Unlike the other methods, the constructor is called on the UI thread. 20 // Unlike the other methods, the constructor is called on the UI thread.
21 SearchProviderInstallStateMessageFilter(int render_process_id, 21 SearchProviderInstallStateMessageFilter(int render_process_id,
22 Profile* profile); 22 Profile* profile);
23 virtual ~SearchProviderInstallStateMessageFilter();
24 23
25 // content::BrowserMessageFilter implementation. 24 // content::BrowserMessageFilter implementation.
26 virtual bool OnMessageReceived(const IPC::Message& message, 25 virtual bool OnMessageReceived(const IPC::Message& message,
27 bool* message_was_ok) OVERRIDE; 26 bool* message_was_ok) OVERRIDE;
28 27
29 private: 28 private:
29 virtual ~SearchProviderInstallStateMessageFilter();
30
30 // Figures out the install state for the search provider. 31 // Figures out the install state for the search provider.
31 search_provider::InstallState GetSearchProviderInstallState( 32 search_provider::InstallState GetSearchProviderInstallState(
32 const GURL& page_location, 33 const GURL& page_location,
33 const GURL& requested_host); 34 const GURL& requested_host);
34 35
35 // Starts handling the message requesting the search provider install state. 36 // Starts handling the message requesting the search provider install state.
36 void OnMsgGetSearchProviderInstallState(const GURL& page_location, 37 void OnMsgGetSearchProviderInstallState(const GURL& page_location,
37 const GURL& requested_host, 38 const GURL& requested_host,
38 IPC::Message* reply_msg); 39 IPC::Message* reply_msg);
39 40
40 // Sends the reply message about the search provider install state. 41 // Sends the reply message about the search provider install state.
41 void ReplyWithProviderInstallState(const GURL& page_location, 42 void ReplyWithProviderInstallState(const GURL& page_location,
42 const GURL& requested_host, 43 const GURL& requested_host,
43 IPC::Message* reply_msg); 44 IPC::Message* reply_msg);
44 45
45 // Used to schedule invocations of ReplyWithProviderInstallState. 46 // Used to schedule invocations of ReplyWithProviderInstallState.
46 base::WeakPtrFactory<SearchProviderInstallStateMessageFilter> weak_factory_; 47 base::WeakPtrFactory<SearchProviderInstallStateMessageFilter> weak_factory_;
47 48
48 // Used to do a load and get information about install states. 49 // Used to do a load and get information about install states.
49 SearchProviderInstallData provider_data_; 50 SearchProviderInstallData provider_data_;
50 51
51 // Copied from the profile since the profile can't be accessed on the I/O 52 // Copied from the profile since the profile can't be accessed on the I/O
52 // thread. 53 // thread.
53 const bool is_off_the_record_; 54 const bool is_off_the_record_;
54 55
55 DISALLOW_COPY_AND_ASSIGN(SearchProviderInstallStateMessageFilter); 56 DISALLOW_COPY_AND_ASSIGN(SearchProviderInstallStateMessageFilter);
56 }; 57 };
57 58
58 #endif // CHROME_BROWSER_SEARCH_ENGINES_SEARCH_PROVIDER_INSTALL_STATE_MESSAGE_F ILTER_H_ 59 #endif // CHROME_BROWSER_SEARCH_ENGINES_SEARCH_PROVIDER_INSTALL_STATE_MESSAGE_F ILTER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698