search

mipengine / mip2

MIP (移动网页加速器)通过优化网页JS、控制资源加载顺序，达到加速网页的效果。
https://www.mipengine.org/
MIT License
184 stars 49 forks source link

fix: Controls video when page is switching #454

Closed yenshih closed 5 years ago

yenshih commented 5 years ago

相关 ISSUE: https://github.com/mipengine/mip2/issues/448

1、升级点

  1. 视频播放时，通过链接打开新的 MIP 页面时（iframe），暂停视频播放
  2. 当通过 cache-first 链接跳回原页面时，重置视频状态到开头

2、影响范围 （描述该需求上线会影响什么功能）

3、自测 Checklist

  1. 暂停视频播放
    • video 页面 (rootPage) -> 其他页面 (iframe) -> video 页面 (iframe)
    • video 页面 (iframe) -> 其他页面 (iframe) -> video 页面 (iframe)
  2. 重置视频状态
    • video 页面 (iframe) -> 其他页面 (iframe) --(cache-first)-> video 页面 (iframe)
    • video 页面 (iframe) -> 其他页面 (iframe) --后退-> video 页面 (iframe)

4、需要覆盖的场景和 Case

5、自测机型和浏览器

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 826

Files with Coverage Reduction New Missed Lines %
src/components/mip-video.js 2 92.66%
<!-- Total: 2 -->
Totals Coverage Status
Change from base Build 836: -0.09%
Covered Lines: 3626
Relevant Lines: 3748
💛 - Coveralls
coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 859

Files with Coverage Reduction New Missed Lines %
src/components/mip-video.js 7 87.85%
<!-- Total: 7 -->
Totals Coverage Status
Change from base Build 836: -0.2%
Covered Lines: 3622
Relevant Lines: 3749
💛 - Coveralls
gaoshan003 commented 5 years ago

测试内容：

Chrome模拟： 打开页面，视频播放正常；异步跳转后，视频停止播放；返回页面，视频及页面展示正常，视频播放正常 http://www.yohoboys.com/mip/news/mobile-35037-0-0.html http://cp01-sys-rpm14.cp01.baidu.com:8003/mip/c/s/www.yohoboys.com/mip/news/mobile-35037-0-0.html 代理测试 http://3g.163.com/sports/article_mip/B2P7NQU500051CAQ.html 其他页面 http://cp01-sys-rpm14.cp01.baidu.com:8003/ala/c/s/m.miaoshou.net/mip/video/646025.html http://cp01-sys-rpm14.cp01.baidu.com:8003/ala/c/s/m.youlai.cn/video/mip/279942.html 使用一台android和一台ios手机的手百和自带浏览器对一视频页面进行测试 http://cp01-sys-rpm14.cp01.baidu.com:8003/ala/c/s/m.youlai.cn/video/mip/279942.html

发现bug 1个 https://github.com/mipengine/mip2/issues/462，已修复 测试通过