Add a comment for the class TabSpecificContentSettings

Add a comment for the class TabSpecificContentSettings


BUG=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=164362

[markusheintz_, 2012-10-25 14:58:00]

I bothers me that the TabSpecificContentSettings class has no comment. So here
is my first attempt.

Comments and suggestion to improve this are appreciated :)

[Bernhard Bauer, 2012-10-25 15:15:27]

lgtm

[plz use chromium.org account, 2012-10-25 16:03:06]

It's not entirely correct, it also tracks if and which content accessed
(not only blocked)

lgtm with that fixed
On Oct 25, 2012 4:58 PM, <markusheintz@chromium.org> wrote:

> Reviewers: bauerb at google, plz use chromium.org account, marja, battre,
>
> Message:
> I bothers me that the TabSpecificContentSettings class has no comment. So
> here
> is my first attempt.
>
> Comments and suggestion to improve this are appreciated :)
>
> Description:
> Add a comment for the class TabSpecificContentSettings
>
>
> BUG=none
>
>
> Please review this at
https://codereview.chromium.**org/11269047/<https://codereview.chromium.org/1...
>
> SVN Base:
svn://svn.chromium.org/chrome/**trunk/src<http://svn.chromium.org/chrome/trunk/src>
>
> Affected files:
>   M chrome/browser/content_**settings/tab_specific_content_**settings.h
>
>
> Index: chrome/browser/content_**settings/tab_specific_content_**settings.h
> diff --git
a/chrome/browser/content_**settings/tab_specific_content_**settings.h
> b/chrome/browser/content_**settings/tab_specific_content_**settings.h
> index 6ad0692caf7e4cdd3ad9b65225418c**7ad20a55e0..**
> c82193116c1ff7a05185e2aeef8b9e**c042b2144b 100644
> --- a/chrome/browser/content_**settings/tab_specific_content_**settings.h
> +++ b/chrome/browser/content_**settings/tab_specific_content_**settings.h
> @@ -34,6 +34,9 @@ namespace net {
>  class CookieOptions;
>  }
>
> +// This class manages state about permissions, content settings, cookies
> and
> +// site data for a specific WebContents. It tracks if content was blocked
> and
> +// provides information about which types of content was blocked.
>  class TabSpecificContentSettings
>      : public content::WebContentsObserver,
>        public content::NotificationObserver,
>
>
>

[markusheintz_, 2012-10-25 16:22:45]

Addressed comments.

[marja, 2012-10-25 16:29:27]

lgtm

[commit-bot: I haz the power, 2012-10-26 07:27:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/markusheintz@chromium.org/11269047/6001

[commit-bot: I haz the power, 2012-10-26 19:29:20]

Change committed as 164362