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

Unified Diff: chrome/browser/search_engines/search_provider_install_state_message_filter.cc

Issue 6683066: Remove render_messages_params files since they're not needed anymore. I moved the search provide... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 9 years, 9 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: chrome/browser/search_engines/search_provider_install_state_message_filter.cc
===================================================================
--- chrome/browser/search_engines/search_provider_install_state_message_filter.cc (revision 79639)
+++ chrome/browser/search_engines/search_provider_install_state_message_filter.cc (working copy)
@@ -7,7 +7,6 @@
#include "base/logging.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/common/render_messages.h"
-#include "chrome/common/render_messages_params.h"
#include "content/browser/renderer_host/render_process_host.h"
#include "content/browser/renderer_host/render_view_host.h"
#include "content/common/notification_source.h"
@@ -49,7 +48,7 @@
return handled;
}
-ViewHostMsg_GetSearchProviderInstallState_Params
+search_provider::InstallState
SearchProviderInstallStateMessageFilter::GetSearchProviderInstallState(
const GURL& page_location,
const GURL& requested_host) {
@@ -57,31 +56,27 @@
// Do the security check before any others to avoid information leaks.
if (page_location.GetOrigin() != requested_origin)
- return ViewHostMsg_GetSearchProviderInstallState_Params::Denied();
+ return search_provider::DENIED;
// In incognito mode, no search information is exposed. (This check must be
// done after the security check or else a web site can detect that the
// user is in incognito mode just by doing a cross origin request.)
if (is_off_the_record_)
- return ViewHostMsg_GetSearchProviderInstallState_Params::NotInstalled();
+ return search_provider::NOT_INSTALLED;
switch (provider_data_.GetInstallState(requested_origin)) {
case SearchProviderInstallData::NOT_INSTALLED:
- return ViewHostMsg_GetSearchProviderInstallState_Params::
- NotInstalled();
+ return search_provider::NOT_INSTALLED;
case SearchProviderInstallData::INSTALLED_BUT_NOT_DEFAULT:
- return ViewHostMsg_GetSearchProviderInstallState_Params::
- InstallButNotDefault();
+ return search_provider::INSTALLED_BUT_NOT_DEFAULT;
case SearchProviderInstallData::INSTALLED_AS_DEFAULT:
- return ViewHostMsg_GetSearchProviderInstallState_Params::
- InstalledAsDefault();
+ return search_provider::INSTALLED_AS_DEFAULT;
}
NOTREACHED();
- return ViewHostMsg_GetSearchProviderInstallState_Params::
- NotInstalled();
+ return search_provider::NOT_INSTALLED;
}
void
@@ -103,7 +98,7 @@
const GURL& requested_host,
IPC::Message* reply_msg) {
DCHECK(reply_msg);
- ViewHostMsg_GetSearchProviderInstallState_Params install_state =
+ search_provider::InstallState install_state =
GetSearchProviderInstallState(page_location, requested_host);
ViewHostMsg_GetSearchProviderInstallState::WriteReplyParams(

Powered by Google App Engine
This is Rietveld 408576698