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

Unified Diff: chrome/browser/content_settings/tab_specific_content_settings.h

Issue 10782038: Revert 147045 - Bring up a content settings icon for ungestured registerProtocolHandler call. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 years, 5 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/content_settings/tab_specific_content_settings.h
===================================================================
--- chrome/browser/content_settings/tab_specific_content_settings.h (revision 147078)
+++ chrome/browser/content_settings/tab_specific_content_settings.h (working copy)
@@ -16,7 +16,6 @@
#include "chrome/browser/geolocation/geolocation_settings_state.h"
#include "chrome/common/content_settings.h"
#include "chrome/common/content_settings_types.h"
-#include "chrome/common/custom_handlers/protocol_handler.h"
#include "content/public/browser/notification_observer.h"
#include "content/public/browser/notification_registrar.h"
#include "content/public/browser/web_contents_observer.h"
@@ -169,41 +168,6 @@
return geolocation_settings_state_;
}
- // Call to indicate that there is a protocol handler pending user approval.
- void set_pending_protocol_handler(const ProtocolHandler& handler) {
- pending_protocol_handler_ = handler;
- }
-
- const ProtocolHandler& pending_protocol_handler() const {
- return pending_protocol_handler_;
- }
-
- void ClearPendingProtocolHandler() {
- pending_protocol_handler_ = ProtocolHandler::EmptyProtocolHandler();
- }
-
-
- // Sets the previous protocol handler which will be replaced by the
- // pending protocol handler.
- void set_previous_protocol_handler(const ProtocolHandler& handler) {
- previous_protocol_handler_ = handler;
- }
-
- const ProtocolHandler& previous_protocol_handler() const {
- return previous_protocol_handler_;
- }
-
- // Set whether the setting for the pending handler is DEFAULT (ignore),
- // ALLOW, or DENY.
- void set_pending_protocol_handler_setting(ContentSetting setting) {
- pending_protocol_handler_setting_ = setting;
- }
-
- ContentSetting pending_protocol_handler_setting() const {
- return pending_protocol_handler_setting_;
- }
-
-
// Returns a pointer to the |LocalSharedObjectsContainer| that contains all
// allowed local shared objects like cookies, local storage, ... .
const LocalSharedObjectsContainer& allowed_local_shared_objects() const {
@@ -315,21 +279,6 @@
// Manages information about Geolocation API usage in this page.
GeolocationSettingsState geolocation_settings_state_;
- // The pending protocol handler, if any. This can be set if
- // registerProtocolHandler was invoked without user gesture.
- // The |IsEmpty| method will be true if no protocol handler is
- // pending registration.
- ProtocolHandler pending_protocol_handler_;
-
- // The previous protocol handler to be replaced by
- // the pending_protocol_handler_, if there is one. Empty if
- // there is no handler which would be replaced.
- ProtocolHandler previous_protocol_handler_;
-
- // The setting on the pending protocol handler registration. Persisted in case
- // the user opens the bubble and makes changes multiple times.
- ContentSetting pending_protocol_handler_setting_;
-
// Stores whether the user can load blocked plugins on this page.
bool load_plugins_link_enabled_;

Powered by Google App Engine
This is Rietveld 408576698