summaryrefslogtreecommitdiffstats
path: root/chrome/browser/content_settings
Commit message (Collapse)AuthorAgeFilesLines
...
* Revert r115276, it broke PanelDownloadTest.Download in interactive_ui_teststhakis@chromium.org2011-12-211-1/+1
| | | | | | | | | | | | | | | | on win,linux,mac. Original change: Rename TabContents::controller() to GetController and put it into the WebContents namespace. BUG=98716 TBR=dpranke Review URL: http://codereview.chromium.org/8956050 TBR=jam Review URL: http://codereview.chromium.org/9018016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115346 0039d316-1c4b-4281-b951-d872f2087c98
* Rename TabContents::controller() to GetController and put it into the ↵jam@chromium.org2011-12-211-1/+1
| | | | | | | | | | WebContents namespace. BUG=98716 TBR=dpranke Review URL: http://codereview.chromium.org/8956050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115276 0039d316-1c4b-4281-b951-d872f2087c98
* Move TabContentsDelegate to content/public/browser and put it in the content ↵tfarina@chromium.org2011-12-201-1/+1
| | | | | | | | | | | | | | namespace. - Rename TabContentsDelegate to WebContentsDelegate and move it to web_contents_delegate.* BUG=98716,105875 R=jam@chromium.org TBR=avi@chromium.org Review URL: http://codereview.chromium.org/9008013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115213 0039d316-1c4b-4281-b951-d872f2087c98
* Split UserMetrics into API vs. implementation. Move API to content/public.joi@chromium.org2011-12-144-6/+10
| | | | | | | | | TBR=davemoore@chromium.org BUG=98716 Review URL: http://codereview.chromium.org/8919017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114416 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up TCW, make it solely a hub for 1:1 observer/helper objects.avi@chromium.org2011-12-121-0/+1
| | | | | | | | | BUG=105872 TEST=no functional change Review URL: http://codereview.chromium.org/8892011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114017 0039d316-1c4b-4281-b951-d872f2087c98
* - Remove unused includesmarkusheintz@chromium.org2011-12-0810-50/+31
| | | | | | | | | | | | | | | - Add missing includes - Inline function. BUG=63656 TEST=HostContentSettingsMapTest*, ContentSettingsProviderTest* Review URL: http://codereview.chromium.org/8804016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113610 0039d316-1c4b-4281-b951-d872f2087c98
* Don't set a default Value for content settigns type AutoSelectCertificate markusheintz@chromium.org2011-12-022-2/+14
| | | | | | | | | BUG=106149 TEST=DefaultProviderTest* Review URL: http://codereview.chromium.org/8776044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112680 0039d316-1c4b-4281-b951-d872f2087c98
* Switch to IWYU for files that previously were getting a bunch ofjoi@chromium.org2011-12-015-0/+5
| | | | | | | | | | | header files via io_thread.h or browser_thread.h TBR=owners (previously approved on codereview 8477004) BUG=98716 Review URL: http://codereview.chromium.org/8746023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112455 0039d316-1c4b-4281-b951-d872f2087c98
* Add UMA for default content settings values, and default content settings ↵markusheintz@chromium.org2011-11-301-9/+46
| | | | | | | | | | | | changed actions. BUG=TODO TEST=none Review URL: http://codereview.chromium.org/8698006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@112182 0039d316-1c4b-4281-b951-d872f2087c98
* Change DefaultRuleIterator to use the Value data type internally instead of ↵markusheintz@chromium.org2011-11-291-16/+10
| | | | | | | | | | | | ContentSetting data type. BUG=102637 TEST=DefaultProviderTest*, HostContentSettingsMapTest* Review URL: http://codereview.chromium.org/8715008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111942 0039d316-1c4b-4281-b951-d872f2087c98
* Remove corrupted values from obsolete geolocation preference.markusheintz@chromium.org2011-11-241-1/+30
| | | | | | | | | | BUG=103272 TEST=No DCHECK, No Crash Review URL: http://codereview.chromium.org/8676017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111518 0039d316-1c4b-4281-b951-d872f2087c98
* Content settings UI for mouse lockscheib@chromium.org2011-11-231-0/+1
| | | | | | | | | | BUG=97768 TEST=Preferences / Under the hood / Content Settings / Mouse Lock should exist and function similar to Location Review URL: http://codereview.chromium.org/8549011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111395 0039d316-1c4b-4281-b951-d872f2087c98
* Don't use the DOMStorageType for a chrome IPC, so that we can get rid of ↵jam@chromium.org2011-11-232-8/+6
| | | | | | | | | that content include from chrome. We change a bool to it, only to change it back to a bool, so no need for it. BUG=98716 Review URL: http://codereview.chromium.org/8667004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111363 0039d316-1c4b-4281-b951-d872f2087c98
* Get rid of a bunch of view_messages.h includes from chrome by making the ↵jam@chromium.org2011-11-212-2/+2
| | | | | | | | | TabContentsObserver interface not use a struct from it. Instead, the interface uses a new struct, content::FrameNavigateParams, that's in the public directory. The IPC message's struct derives from it. BUG=98716 Review URL: http://codereview.chromium.org/8603015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110963 0039d316-1c4b-4281-b951-d872f2087c98
* Add OVERRIDE to chrome/browser/.avi@chromium.org2011-11-211-1/+1
| | | | | | | | | BUG=104314 TEST=no change Review URL: http://codereview.chromium.org/8612007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110939 0039d316-1c4b-4281-b951-d872f2087c98
* Fix memory leak in the SetDefaultContentSettings method of the ↵markusheintz@chromium.org2011-11-212-11/+17
| | | | | | | | | | | HostContentSettingaMap. BUG=104775, 102637 TEST=HostContentSettingsMapTest* Review URL: http://codereview.chromium.org/8528031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110908 0039d316-1c4b-4281-b951-d872f2087c98
* Add a section for the fullscreen JS API in the content settings page.koz@chromium.org2011-11-181-5/+11
| | | | | | | | | BUG=100292 Review URL: http://codereview.chromium.org/8585047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110654 0039d316-1c4b-4281-b951-d872f2087c98
* Valgrind: Fix a leak in ContentSettingsProvider.jhawkins@chromium.org2011-11-172-7/+6
| | | | | | | | | | | | | | If the call to SetWebsiteSetting() fails, the value is still owned by the caller and must be deleted. BUG=none TEST=none R=bauerb@chromium.org Review URL: http://codereview.chromium.org/8592001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110594 0039d316-1c4b-4281-b951-d872f2087c98
* Remove some unnecessary includes of view_messages.h.jam@chromium.org2011-11-171-1/+0
| | | | | | | | BUG=98716 TBR=ananta Review URL: http://codereview.chromium.org/8590033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110522 0039d316-1c4b-4281-b951-d872f2087c98
* Replace SetContentSetting method of the content_settings::Provider interface ↵markusheintz@chromium.org2011-11-1717-211/+336
| | | | | | | | | | | | | | | | | with GetWebsiteSetting. This is part of a series of cleanup CLs to make all content settings Providers and the HostContentSetting map use only |Values| instead of |ContentSettings|. The HostContentSettingsMap should only contain conveniences methods that take |ContentSetting| parameters. BUG=102637 TEST=HostContentSettingsMapTest*, PrefProviderTest*, PolicyProviderTest*, DefaultProviderTest*, ExtensionProviderTest* Review URL: http://codereview.chromium.org/8539004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110500 0039d316-1c4b-4281-b951-d872f2087c98
* chrome: Remove 14 exit time destructors and 2 static initializers.thakis@chromium.org2011-11-161-1/+1
| | | | | | | | | | | | BUG=101600, 94925 TEST=none TBR=mnissler Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=110235 Review URL: http://codereview.chromium.org/8573021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110330 0039d316-1c4b-4281-b951-d872f2087c98
* Take script URLs into account when applying script content settings.marja@chromium.org2011-11-1611-173/+101
| | | | | | | | | | | | | | | | | | Transmit script content settings to the renderer. Use the script URL as the secondary URL for the content setting rules. This CL contains the functionality of the following 2 CLs, hopefully without performance regressions: http://codereview.chromium.org/8409006 http://codereview.chromium.org/8498007 BUG=90840 TEST=ChromeRenderViewTest.ContentSettings(Allow|Block)Scripts Review URL: http://codereview.chromium.org/8538004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110286 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 110235 - chrome: Remove 14 exit time destructors and 2 static ↵kmadhusu@chromium.org2011-11-161-1/+1
| | | | | | | | | | | | | | | initializers. BUG=101600, 94925 TEST=none TBR=mnissler Review URL: http://codereview.chromium.org/8573021 TBR=thakis@chromium.org Review URL: http://codereview.chromium.org/8568039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110238 0039d316-1c4b-4281-b951-d872f2087c98
* chrome: Remove 14 exit time destructors and 2 static initializers.thakis@chromium.org2011-11-161-1/+1
| | | | | | | | | | BUG=101600, 94925 TEST=none TBR=mnissler Review URL: http://codereview.chromium.org/8573021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110235 0039d316-1c4b-4281-b951-d872f2087c98
* Allow linker initialization of lazy instancejoth@chromium.org2011-11-151-2/+2
| | | | | | | | | | | | | | Using the initializer list construct = {0} allows the object to be linker initialized. Modify the LazyInstance class design to make it a pod aggregate type that can be linker initialized this way. Also combines the instance and state members, in line with the Singleton<> class design. Introduces a new LAZY_INSTANCE_INITIALIZER macro specifically for using to init all lazy instances + modify all existing callsites to use it. (Old code would no longer compile) BUG=94925 TEST=existing tests pass. http://build.chromium.org/f/chromium/perf/linux-release/sizes/report.html?history=150&header=chrome-si&graph=chrome-si&rev=-1 should step downward. TBR=jam@chromium.org,rvargas@chromium.org,darin@chromium.org,ben@chromium.org,apatrick@chromium.org,akalin@chromium.org Review URL: http://codereview.chromium.org/8491043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110076 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup: Remove unneeded forward declarations from various chrome/browser ↵thestig@chromium.org2011-11-151-1/+0
| | | | | | | | | | | subdirectories. BUG=none TEST=none Review URL: http://codereview.chromium.org/8539006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109987 0039d316-1c4b-4281-b951-d872f2087c98
* Switch BackgroundContents to use TabContents instead of RenderViewHost.jam@chromium.org2011-11-112-2/+2
| | | | | | | BUG=99698, 98934 Review URL: http://codereview.chromium.org/8469015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109661 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 109213 - Take script URLs into account when applying script content ↵cmp@chromium.org2011-11-103-20/+3
| | | | | | | | | | | | | | | | | | | | | | | | | settings. <cmp> We suspect this CL caused a performance regression on chromium.perf. Reverting to see if this is the source of the regression. Transmit script content settings to the renderer. Use the script URL as the secondary URL for the content setting rules. BUG=90840 TEST=ChromeRenderViewTest.ContentSettings(Allow|Block)Scripts Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=109005 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=109036 Review URL: http://codereview.chromium.org/8409006 TBR=marja@chromium.org Review URL: http://codereview.chromium.org/8510040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109362 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 109228 - ContentSettingsObserver (+ related classes) cleanup.cmp@chromium.org2011-11-109-82/+170
| | | | | | | | | | | | | | | | | | | <cmp> This CL needs to be reverted before we revert 109213, which we speculate caused some performance regressions. Per-page content settings and default content settings are no longer needed, since the renderer stores the content setting rules for images and scripts. BUG=102662 TEST=NONE Review URL: http://codereview.chromium.org/8498007 TBR=marja@chromium.org Review URL: http://codereview.chromium.org/8497062 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109360 0039d316-1c4b-4281-b951-d872f2087c98
* Don't whitelist extensions for geolocation.bauerb@chromium.org2011-11-091-1/+3
| | | | | | | | | | BUG=103530 TEST=see bug Review URL: http://codereview.chromium.org/8510008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109255 0039d316-1c4b-4281-b951-d872f2087c98
* ContentSettingsObserver (+ related classes) cleanup.marja@chromium.org2011-11-099-170/+82
| | | | | | | | | | | | | Per-page content settings and default content settings are no longer needed, since the renderer stores the content setting rules for images and scripts. BUG=102662 TEST=NONE Review URL: http://codereview.chromium.org/8498007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109228 0039d316-1c4b-4281-b951-d872f2087c98
* Take script URLs into account when applying script content settings.marja@chromium.org2011-11-093-3/+20
| | | | | | | | | | | | | | | | | Transmit script content settings to the renderer. Use the script URL as the secondary URL for the content setting rules. BUG=90840 TEST=ChromeRenderViewTest.ContentSettings(Allow|Block)Scripts Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=109005 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=109036 Review URL: http://codereview.chromium.org/8409006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109213 0039d316-1c4b-4281-b951-d872f2087c98
* Lock the GTK UI of the content settings bubble if the corresponding content ↵markusheintz@chromium.org2011-11-084-56/+94
| | | | | | | | | | | | settings type is managed. BUG=98029 TEST=HostContentSettingsMap* Review URL: http://codereview.chromium.org/8493013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109068 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 109036 (speculative; seems to cause increased worker test flakiness) ↵thakis@chromium.org2011-11-083-20/+3
| | | | | | | | | | | | | | | | | | | | - Take script URLs into account when applying script content settings. Transmit script content settings to the renderer. Use the script URL as the secondary URL for the content setting rules. BUG=90840 TEST=ChromeRenderViewTest.ContentSettings(Allow|Block)Scripts Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=109005 Review URL: http://codereview.chromium.org/8409006 TBR=marja@chromium.org Review URL: http://codereview.chromium.org/8499023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109044 0039d316-1c4b-4281-b951-d872f2087c98
* Take script URLs into account when applying script content settings.marja@chromium.org2011-11-083-3/+20
| | | | | | | | | | | | | | | Transmit script content settings to the renderer. Use the script URL as the secondary URL for the content setting rules. BUG=90840 TEST=ChromeRenderViewTest.ContentSettings(Allow|Block)Scripts Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=109005 Review URL: http://codereview.chromium.org/8409006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109036 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 109005 - Take script URLs into account when applying script content ↵marja@chromium.org2011-11-083-20/+3
| | | | | | | | | | | | | | | | | | settings. Transmit script content settings to the renderer. Use the script URL as the secondary URL for the content setting rules. BUG=90840 TEST=ChromeRenderViewTest.ContentSettings(Allow|Block)Scripts Review URL: http://codereview.chromium.org/8409006 TBR=marja@chromium.org Review URL: http://codereview.chromium.org/8502014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109014 0039d316-1c4b-4281-b951-d872f2087c98
* Take script URLs into account when applying script content settings.marja@chromium.org2011-11-083-3/+20
| | | | | | | | | | | | | Transmit script content settings to the renderer. Use the script URL as the secondary URL for the content setting rules. BUG=90840 TEST=ChromeRenderViewTest.ContentSettings(Allow|Block)Scripts Review URL: http://codereview.chromium.org/8409006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@109005 0039d316-1c4b-4281-b951-d872f2087c98
* Make third-party cookie blocking strict by defaultjochen@chromium.org2011-11-072-16/+8
| | | | | | | | | | BUG=98241 TEST=CookieSettings.* Review URL: http://codereview.chromium.org/8479027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108846 0039d316-1c4b-4281-b951-d872f2087c98
* Exempt chrome-extension:// from third-party cookie rules, but not from ↵jochen@chromium.org2011-11-044-10/+50
| | | | | | | | | | | | regular content settings BUG=99588 TEST=CookieSettingsTest Review URL: http://codereview.chromium.org/8462003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108664 0039d316-1c4b-4281-b951-d872f2087c98
* Apply exemptions to content settings based on the primary urljochen@chromium.org2011-11-043-3/+25
| | | | | | | | | | | the primary url is the cookie's security origin or, for all other content types, the main frame url BUG=none TEST=HostContentSettingsMapTest.ShouldAllowAllContent Review URL: http://codereview.chromium.org/8457009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108644 0039d316-1c4b-4281-b951-d872f2087c98
* Remove needless include of the content/browser/render_process_host.h header ↵ananta@chromium.org2011-11-041-1/+0
| | | | | | | | | | | file. Part 1 of upcoming changes to move RPH/BRPH to a public interface. BUG=98716 Review URL: http://codereview.chromium.org/8451012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108608 0039d316-1c4b-4281-b951-d872f2087c98
* Move BrowserThread to content namespace.joi@chromium.org2011-11-0212-0/+21
| | | | | | | | | TBR=owners BUG=98716 Review URL: http://codereview.chromium.org/8437002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108270 0039d316-1c4b-4281-b951-d872f2087c98
* Split BrowserThread into public API and private implementation, step 1.joi@chromium.org2011-10-2811-16/+18
| | | | | | | | | | | | | | | | | | | | | Only content/ now has the ability to create BrowserThread objects, with the exception that tests can create the content::TestBrowserThread subclass, and (temporarily) code in chrome/ can create the DeprecatedBrowserThread subclass. A follow-up change will make content/ take care of its own thread creation, remove DeprecatedBrowserThread, and move all state and non-trivial constructors from BrowserThread down to BrowserThreadImpl. Also moved BrowserProcessSubThread into content/ namespace. As part of follow-up cleanup, chrome/ will stop using this class. BUG=98716 TEST=existing Review URL: http://codereview.chromium.org/8392042 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107718 0039d316-1c4b-4281-b951-d872f2087c98
* Adding CookieSettings for storing cookie content settings.marja@chromium.org2011-10-289-445/+698
| | | | | | | | | | | | | | | | Moving cookie-spesific logic from HostContentSettingsMap to CookieSettings. Making it explicit that the "is cookie session only" decision only depends on the primary url. Also removes the migration code for the obsolete kCookieBehavior preference. BUG=93335 TEST=CookieSettings.* Review URL: http://codereview.chromium.org/8383004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107705 0039d316-1c4b-4281-b951-d872f2087c98
* Delegating the "are images allowed" decision to renderer.marja@chromium.org2011-10-274-21/+12
| | | | | | | | | | | | This enables making the decision based on both image url and the page url. E.g., blocking third-party images. BUG=81179 TEST=RenderViewTest.ImagesBlockedByDefault, RenderViewTest.ImagesAllowedByDefault Review URL: http://codereview.chromium.org/7831075 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107562 0039d316-1c4b-4281-b951-d872f2087c98
* Mouse lock content setting back-end (no UI)scheib@chromium.org2011-10-223-1/+9
| | | | | | | | | | | | BUG=100677 TEST=Manual test in ppapi/examples/mouse_lock; enter full screen and mouse lock; prompts should appear only first time. R=jochen@chromium.org, pkasting@chromium.org, yzshen@chromium.org Review URL: http://codereview.chromium.org/8354026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106821 0039d316-1c4b-4281-b951-d872f2087c98
* Fixing the "lock is not held" assertions in PrefProvider.marja@chromium.org2011-10-202-48/+44
| | | | | | | | | | | | | It is ok if another thread holds the lock; what we want to assert is that this thread doesn't hold it. BUG=NONE TEST=NONE Review URL: http://codereview.chromium.org/8342071 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106512 0039d316-1c4b-4281-b951-d872f2087c98
* Make NotificationService an interface in the content namespace, and switch ↵jam@chromium.org2011-10-196-20/+20
| | | | | | | | | callers to use it. Move the implementation to content/browser. Stop creating it in all child processes since it's only used in the browser. BUG=98716 Review URL: http://codereview.chromium.org/8342048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106403 0039d316-1c4b-4281-b951-d872f2087c98
* Take the content settings set by extensions into account in the default ↵marja@chromium.org2011-10-193-65/+61
| | | | | | | | | | | | | | | setting UI. If an extension sets a (wildcard, wildcard) content setting, disable the default setting UI and display a banner. BUG=NONE TEST=manual Review URL: http://codereview.chromium.org/8333015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106289 0039d316-1c4b-4281-b951-d872f2087c98
* Check for default content setting pattern when requiring user authorization ↵bauerb@chromium.org2011-10-194-108/+105
| | | | | | | | | | | | | | for plug-ins. We now check in the renderer if a plug-in is allowed by default, which is the case if the matching patterns are wildcards. BUG=100581 TEST=see bug Review URL: http://codereview.chromium.org/8334020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106270 0039d316-1c4b-4281-b951-d872f2087c98