Don't use web intents for rss feeds.

Don't use web intents for rss feeds.


R=jhawkins@chromium.org
BUG=156373
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=161675

[James Hawkins, 2012-10-11 23:08:02]

LGTM pending try jobs.

[Greg Billock, 2012-10-11 23:12:43]

On 2012/10/11 23:08:02, James Hawkins wrote:
> LGTM pending try jobs.

can I get owners lgtm for this?

[Paweł Hajdan Jr., 2012-10-11 23:35:08]

chrome/browser/download LGTM

[Greg Billock, 2012-10-11 23:37:52]

On 2012/10/11 23:35:08, Paweł Hajdan Jr. wrote:
> chrome/browser/download LGTM

Scott, Will, can you lgtm for owners?

(picking manually better than the auto-suggest...)

[willchan no longer on Chromium, 2012-10-11 23:53:30]

Seems fine, but why not rss? Is there an explanation somewhere?

[willchan no longer on Chromium, 2012-10-11 23:54:19]

Also we should figure out how to proceed in the future so y'all don't have to
keep changing net/ for this. Maybe you guys should create your own list
elsewhere?

[sky, 2012-10-11 23:57:37]

LGTM

[Greg Billock, 2012-10-12 00:04:54]

On 2012/10/11 23:54:19, willchan wrote:
> Also we should figure out how to proceed in the future so y'all don't have to
> keep changing net/ for this. Maybe you guys should create your own list
> elsewhere?

There's widespread dissatisfaction with not having a good built-in view option,
so we're going to make that and re-enable.

The reason this list needs changed is that it's the list of
browser-native-supported mime types. Arguably that should be in chrome/ not
net/, but right now it's not.

[willchan no longer on Chromium, 2012-10-12 16:41:02]

OK, LGTM


On Thu, Oct 11, 2012 at 5:04 PM, <gbillock@chromium.org> wrote:

> On 2012/10/11 23:54:19, willchan wrote:
>
>> Also we should figure out how to proceed in the future so y'all don't
>> have to
>> keep changing net/ for this. Maybe you guys should create your own list
>> elsewhere?
>>
>
> There's widespread dissatisfaction with not having a good built-in view
> option,
> so we're going to make that and re-enable.
>
> The reason this list needs changed is that it's the list of
> browser-native-supported mime types. Arguably that should be in chrome/ not
> net/, but right now it's not.
>
>
>
http://codereview.chromium.**org/11048039/<http://codereview.chromium.org/110...
>

[commit-bot: I haz the power, 2012-10-12 16:49:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gbillock@chromium.org/11048039/2003

[Greg Billock, 2012-10-12 21:55:19]

On 2012/10/12 16:49:43, I haz the power (commit-bot) wrote:
> CQ is trying da patch. Follow status at
> https://chromium-status.appspot.com/cq/gbillock%40chromium.org/11048039/2003

asanka? I'd like to submit this today.

[asanka, 2012-10-12 22:46:46]

LGTM

[commit-bot: I haz the power, 2012-10-12 23:37:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/gbillock@chromium.org/11048039/2003

[commit-bot: I haz the power, 2012-10-12 23:40:07]

Change committed as 161675