Use effectivePreloadType() where possible

Use effectivePreloadType() where possible

preloadType() is still used in two places where it ideally should not,
HTMLMediaElement::setPlayerPreload and HTMLMediaElement::seek, but
changing those would probably be observable and need tests.

Drive-by: Drop two unused HTMLMediaElement state bits

R=fs@opera.com

Committed: https://crrev.com/d48519f92275c1946aaf3c66ca6f144d6d2bd3c2
Cr-Commit-Position: refs/heads/master@{#374897}

[philipj_slow, 2016-02-10 04:58:48]

The CQ bit was checked by philipj@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-02-10 04:59:23]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1687793002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1687793002/1

[philipj_slow, 2016-02-10 05:05:46]

Description was changed from

==========
Use effectivePreloadType() where possible

preloadType() is still used in two places where it ideally should not,
HTMLMediaElement::setPlayerPreload and HTMLMediaElement::seek, but
changing those would probably be observable and need tests.

Drive-by: m_delayingLoadForPreloadNone was unused

R=fs@opera.com
==========

to

==========
Use effectivePreloadType() where possible

preloadType() is still used in two places where it ideally should not,
HTMLMediaElement::setPlayerPreload and HTMLMediaElement::seek, but
changing those would probably be observable and need tests.

Drive-by: Drop two unused HTMLMediaElement state bits

R=fs@opera.com
BUG=
==========

[fs, 2016-02-10 08:50:07]

lgtm

[Srirama, 2016-02-10 11:57:02]

srirama.m@samsung.com changed reviewers:
+ srirama.m@samsung.com

[Srirama, 2016-02-10 11:57:02]

Do we need "BUG=" in description?

[fs, 2016-02-10 11:57:32]

Description was changed from

==========
Use effectivePreloadType() where possible

preloadType() is still used in two places where it ideally should not,
HTMLMediaElement::setPlayerPreload and HTMLMediaElement::seek, but
changing those would probably be observable and need tests.

Drive-by: Drop two unused HTMLMediaElement state bits

R=fs@opera.com
BUG=
==========

to

==========
Use effectivePreloadType() where possible

preloadType() is still used in two places where it ideally should not,
HTMLMediaElement::setPlayerPreload and HTMLMediaElement::seek, but
changing those would probably be observable and need tests.

Drive-by: Drop two unused HTMLMediaElement state bits

R=fs@opera.com
==========

[fs, 2016-02-10 11:57:59]

On 2016/02/10 at 11:57:02, srirama.m wrote:
> Do we need "BUG=" in description?

No, but I think the upload script adds it.

[fs, 2016-02-10 11:58:08]

On 2016/02/10 at 11:57:59, fs wrote:
> On 2016/02/10 at 11:57:02, srirama.m wrote:
> > Do we need "BUG=" in description?
> 
> No, but I think the upload script adds it.

(I removed it)

[philipj_slow, 2016-02-11 14:32:47]

On 2016/02/10 11:58:08, fs wrote:
> On 2016/02/10 at 11:57:59, fs wrote:
> > On 2016/02/10 at 11:57:02, srirama.m wrote:
> > > Do we need "BUG=" in description?
> > 
> > No, but I think the upload script adds it.
> 
> (I removed it)

Oh, the shame! I try to always remove it myself :)

[philipj_slow, 2016-02-11 14:32:50]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2016-02-11 14:33:03]

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

[commit-bot: I haz the power, 2016-02-11 15:39:00]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-16 22:35:36]

Description was changed from

==========
Use effectivePreloadType() where possible

preloadType() is still used in two places where it ideally should not,
HTMLMediaElement::setPlayerPreload and HTMLMediaElement::seek, but
changing those would probably be observable and need tests.

Drive-by: Drop two unused HTMLMediaElement state bits

R=fs@opera.com
==========

to

==========
Use effectivePreloadType() where possible

preloadType() is still used in two places where it ideally should not,
HTMLMediaElement::setPlayerPreload and HTMLMediaElement::seek, but
changing those would probably be observable and need tests.

Drive-by: Drop two unused HTMLMediaElement state bits

R=fs@opera.com

Committed: https://crrev.com/d48519f92275c1946aaf3c66ca6f144d6d2bd3c2
Cr-Commit-Position: refs/heads/master@{#374897}
==========

[commit-bot: I haz the power, 2016-02-16 22:35:36]

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