Remove http/tests/plugins/interrupted-get-url.html

Remove http/tests/plugins/interrupted-get-url.html

This test exercises NPAPI functionality that has no equivalent in
PPAPI. Also removes a bunch of unreferenced test resources.

BUG=493212
Committed: https://crrev.com/f1164e96ddf077c6272935a58efc3ea7ee983fdc
Cr-Commit-Position: refs/heads/master@{#378165}

[dcheng, 2016-02-28 08:24:45]

dcheng@chromium.org changed reviewers:
+ raymes@chromium.org, wfh@chromium.org

[dcheng, 2016-02-28 20:20:09]

Description was changed from

==========
Remove http/tests/plugins layout tests.

These tests exercise NPAPI functionality that has no equivalent in
PPAPI:
- PPAPI plugins can't directly script the page.
- There is no equivalent of NPN_GetURL in PPAPI.

BUG=493212
==========

to

==========
Remove http/tests/plugins/interrupted-get-url.html

These test exercises NPAPI functionality that has no equivalent in
PPAPI.

BUG=493212
==========

[dcheng, 2016-02-28 20:20:32]

Description was changed from

==========
Remove http/tests/plugins/interrupted-get-url.html

These test exercises NPAPI functionality that has no equivalent in
PPAPI.

BUG=493212
==========

to

==========
Remove http/tests/plugins/interrupted-get-url.html

This test exercises NPAPI functionality that has no equivalent in
PPAPI. Also removes a bunch of unreferenced test resources.

BUG=493212
==========

[dcheng, 2016-02-28 20:20:40]

dcheng@chromium.org changed reviewers:
- wfh@chromium.org

[dcheng, 2016-02-28 20:20:46]

The CQ bit was checked by dcheng@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-02-28 20:20:53]

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

[commit-bot: I haz the power, 2016-02-28 21:32:36]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-02-28 21:32:37]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[raymes, 2016-02-28 23:42:44]

lg to me but I don't really know anything about these tests. Is there someone
who would have more context? 

Did these start to fail when you removed some other piece of NPAPI code?

Thanks for working on this though :)

[dcheng, 2016-02-28 23:46:51]

On 2016/02/28 at 23:42:44, raymes wrote:
> lg to me but I don't really know anything about these tests. Is there someone
who would have more context? 
> 
> Did these start to fail when you removed some other piece of NPAPI code?
> 
> Thanks for working on this though :)

No, this is mostly just cleanup. Somewhat surprisingly, interrupted-get-url.html
isn't failing even though NPN_GetURL has already been removed… so it's not
likely they're not really testing all that much to begin with.

[Agc, 2016-02-28 23:53:49]

On 2016/02/28 23:46:51, dcheng wrote:
> On 2016/02/28 at 23:42:44, raymes wrote:
> > lg to me but I don't really know anything about these tests. Is there
someone
> who would have more context? 
> > 
> > Did these start to fail when you removed some other piece of NPAPI code?
> > 
> > Thanks for working on this though :)
> 
> No, this is mostly just cleanup. Somewhat surprisingly,
interrupted-get-url.html
> isn't failing even though NPN_GetURL has already been removed… so it's not
> likely they're not really testing all that much to begin with.

http://www.macofalltrades.com/Articles.asp?

[raymes, 2016-02-28 23:54:27]

raymes@chromium.org changed reviewers:
+ agc198115@gmail.com
- Agc198115@gmail.com

[raymes, 2016-02-28 23:54:27]

lgtm

[dcheng, 2016-02-28 23:55:58]

The CQ bit was checked by dcheng@chromium.org

[commit-bot: I haz the power, 2016-02-28 23:56:08]

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

[commit-bot: I haz the power, 2016-02-29 01:58:40]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-02-29 01:59:59]

Description was changed from

==========
Remove http/tests/plugins/interrupted-get-url.html

This test exercises NPAPI functionality that has no equivalent in
PPAPI. Also removes a bunch of unreferenced test resources.

BUG=493212
==========

to

==========
Remove http/tests/plugins/interrupted-get-url.html

This test exercises NPAPI functionality that has no equivalent in
PPAPI. Also removes a bunch of unreferenced test resources.

BUG=493212

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

[commit-bot: I haz the power, 2016-02-29 02:00:01]

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

[agc198115_gmail.com, 2016-02-29 07:33:21]

tel:3975623

El El Dim, 28 de Fev de 2016 a las 18:00, commit-bot@chromium.org via
codereview.chromium.org <reply@chromiumcodereview-hr.appspotmail.com>
escribió:

> Patchset 3 (id:??) landed as
> https://crrev.com/f1164e96ddf077c6272935a58efc3ea7ee983fdc
> Cr-Commit-Position: refs/heads/master@{#378165}
>
> https://codereview.chromium.org/1747713002/
>

-- 
You received this message because you are subscribed to the Google Groups "Blink
Reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

[agc198115_gmail.com, 2016-02-29 07:33:21]

tel:3975623

El El Dim, 28 de Fev de 2016 a las 18:00, commit-bot@chromium.org via
codereview.chromium.org <reply@chromiumcodereview-hr.appspotmail.com>
escribió:

> Patchset 3 (id:??) landed as
> https://crrev.com/f1164e96ddf077c6272935a58efc3ea7ee983fdc
> Cr-Commit-Position: refs/heads/master@{#378165}
>
> https://codereview.chromium.org/1747713002/
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.