Updated the <webview> loadabort documentatin with the actual "reason" strings that are currently us…

Updated the <webview> loadabort documentatin with the actual "reason" strings that are currently used.

BUG=470479
Committed: https://crrev.com/d497f00129fdad062e37eaf5726edda76b2bd212
Cr-Commit-Position: refs/heads/master@{#325239}

[paulmeyer, 2015-04-14 20:33:04]

paulmeyer@chromium.org changed reviewers:
+ lazyboy@chromium.org

[paulmeyer, 2015-04-14 20:33:05]

+lazyboy@

[lazyboy, 2015-04-15 01:58:45]

Thanks for fixing this.

Although this is not the whole set e.reason can take, there can be more, right?
Regardless this is an improvement, so LGTM.

[paulmeyer, 2015-04-15 12:27:00]

On 2015/04/15 01:58:45, lazyboy wrote:
> Thanks for fixing this.
> 
> Although this is not the whole set e.reason can take, there can be more,
right?
> Regardless this is an improvement, so LGTM.

I could not find a complete list, but these are all the ones I could find
reference to. Are you aware of any others?

[lazyboy, 2015-04-15 12:34:18]

On 2015/04/15 12:27:00, Paul Meyer wrote:
> On 2015/04/15 01:58:45, lazyboy wrote:
> > Thanks for fixing this.
> > 
> > Although this is not the whole set e.reason can take, there can be more,
> right?
> > Regardless this is an improvement, so LGTM.
> 
> I could not find a complete list, but these are all the ones I could find
> reference to. Are you aware of any others?

In theory, couldn't it be any one from the net_error_list.h?
But that's too long. What you have right now are the most frequent ones
we encounter I believe, which seems fine. Maybe the description field
"String indicating what type of abort occurred." can hint that this list
is not complete.

[paulmeyer, 2015-04-15 13:40:46]

On 2015/04/15 12:34:18, lazyboy wrote:
> On 2015/04/15 12:27:00, Paul Meyer wrote:
> > On 2015/04/15 01:58:45, lazyboy wrote:
> > > Thanks for fixing this.
> > > 
> > > Although this is not the whole set e.reason can take, there can be more,
> > right?
> > > Regardless this is an improvement, so LGTM.
> > 
> > I could not find a complete list, but these are all the ones I could find
> > reference to. Are you aware of any others?
> 
> In theory, couldn't it be any one from the net_error_list.h?
> But that's too long. What you have right now are the most frequent ones
> we encounter I believe, which seems fine. Maybe the description field
> "String indicating what type of abort occurred." can hint that this list
> is not complete.

I didn't think that all of the errors could be responsible for a loadabort
specifically, but I'm not sure of that, and also just in case I've missed some,
I've added a disclaimer to the description.

[paulmeyer, 2015-04-15 13:41:00]

The CQ bit was checked by paulmeyer@chromium.org

[paulmeyer, 2015-04-15 13:41:00]

The patchset sent to the CQ was uploaded after l-g-t-m from lazyboy@chromium.org
Link to the patchset: https://chromiumcodereview.appspot.com/1065873004/#ps20001
(title: "Rebased; Addressed comment.")

[commit-bot: I haz the power, 2015-04-15 13:41:06]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1065873004/20001

[commit-bot: I haz the power, 2015-04-15 14:04:33]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-04-15 14:05:22]

Patchset 2 (id:??) landed as
https://crrev.com/d497f00129fdad062e37eaf5726edda76b2bd212
Cr-Commit-Position: refs/heads/master@{#325239}