Fix webview tag's documentation to specify correct ClearDataTypeSet for clearData API.

Fix webview tag's documentation to specify correct ClearDataTypeSet for clearData API.

BUG=371488
NOTRY=true
Test=The preview link for this patch is:
https://chrome-apps-doc.appspot.com/_patch/271913003/apps/tags/webview#type-ClearDataTypeSet

Once the change is committed, this can be viewed in 
https://developer.chrome.com/apps/tags/webview#type-ClearDataTypeSet

You should see only 6 items listed as types:
appcache/cookies/fileSystems/indexedDB/localStorage/webSQL
And it should not list unsupported items such as "downloads", "pluginData", ... which it does without this change.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=269440

[scheib, 2014-05-08 20:44:23]

I'm happy to offer OWNERS approval here - but would like an lgtm from someone
familiar with webview API.

The TEST= field in change comments is intended for the quality assurance team
when as they work through a Chrome release.
https://sites.google.com/a/chromium.org/dev/developers/contributing-code?pli=...


Documentation changes:
The preview link for this patch is:
https://chrome-apps-doc.appspot.com/_patch/271913003/apps/tags/webview#type-C...
The internal team documentation has reminders for this:
https://sites.google.com/a/google.com/chrome-apps/development/how-to-work-on-...

[scheib, 2014-05-08 20:45:15]

On 2014/05/08 20:44:23, scheib wrote:
> I'm happy to offer OWNERS approval here - but would like an lgtm from someone
> familiar with webview API.
> 
> The TEST= field in change comments is intended for the quality assurance team
> when as they work through a Chrome release.
>
https://sites.google.com/a/chromium.org/dev/developers/contributing-code?pli=...
> 
> 
> Documentation changes:
> The preview link for this patch is:
>
https://chrome-apps-doc.appspot.com/_patch/271913003/apps/tags/webview#type-C...
> The internal team documentation has reminders for this:
>
https://sites.google.com/a/google.com/chrome-apps/development/how-to-work-on-...

Hmm - the docserver may not be showing that correctly though? I don't see those
sections removed -- if it works locally but not on server please notify
chrome-apps-team & kalman.

[Fady Samuel, 2014-05-08 21:26:51]

webview API lgtm

[scheib, 2014-05-08 21:35:07]

lgtm (intended this time ;)). Please fix up the change description.

[lazyboy, 2014-05-08 21:58:35]

On 2014/05/08 20:45:15, scheib wrote:
> On 2014/05/08 20:44:23, scheib wrote:
> > I'm happy to offer OWNERS approval here - but would like an lgtm from
someone
> > familiar with webview API.
> > 
> > The TEST= field in change comments is intended for the quality assurance
team
> > when as they work through a Chrome release.
> >
>
https://sites.google.com/a/chromium.org/dev/developers/contributing-code?pli=...
> > 
> > 
> > Documentation changes:
> > The preview link for this patch is:
> >
>
https://chrome-apps-doc.appspot.com/_patch/271913003/apps/tags/webview#type-C...
> > The internal team documentation has reminders for this:
> >
>
https://sites.google.com/a/google.com/chrome-apps/development/how-to-work-on-...
Changelist descriptoin updated.
> 
> Hmm - the docserver may not be showing that correctly though? I don't see
those
> sections removed -- if it works locally but not on server please notify
> chrome-apps-team & kalman.
Sent email, it is indeed working locally but not through
chrome-apps-doc.appspot.com

[lazyboy, 2014-05-08 22:30:34]

The CQ bit was checked by lazyboy@chromium.org

[commit-bot: I haz the power, 2014-05-08 22:31:58]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/lazyboy@chromium.org/271913003/1

[commit-bot: I haz the power, 2014-05-09 22:34:49]

Change committed as 269440