Merge lp:~nick-dedekind/unity8/inline-dash-videos into lp:unity8
| Status: | Superseded | ||||
|---|---|---|---|---|---|
| Proposed branch: | lp:~nick-dedekind/unity8/inline-dash-videos | ||||
| Merge into: | lp:unity8 | ||||
| Diff against target: |
2432 lines (+2089/-59) 20 files modified
qml/Components/LazyImage.qml (+2/-0) qml/Components/MediaServices/MediaServices.qml (+391/-0) qml/Components/MediaServices/MediaServicesControls.qml (+189/-0) qml/Components/MediaServices/MediaServicesHeader.qml (+72/-0) qml/Components/MediaServices/VideoPlayer.qml (+152/-0) qml/Components/MediaServices/VideoPlayerControls.qml (+182/-0) qml/Dash/GenericScopeView.qml (+1/-0) qml/Dash/Previews/PreviewInlineVideo.qml (+89/-0) qml/Dash/Previews/PreviewWidgetFactory.qml (+8/-1) tests/mocks/QtMultimedia/CMakeLists.txt (+8/-2) tests/mocks/QtMultimedia/VideoSurface.qml (+111/-0) tests/mocks/QtMultimedia/mediaplayer.cpp (+216/-29) tests/mocks/QtMultimedia/mediaplayer.h (+131/-10) tests/mocks/QtMultimedia/plugin.cpp (+7/-3) tests/mocks/QtMultimedia/videooutput.cpp (+127/-0) tests/mocks/QtMultimedia/videooutput.h (+52/-0) tests/qmltests/CMakeLists.txt (+2/-0) tests/qmltests/Components/tst_MediaServices.qml (+228/-0) tests/qmltests/Dash/Previews/tst_PreviewInlineVideo.qml (+106/-0) tests/qmltests/Dash/Previews/tst_PreviewWidgetFactory.qml (+15/-14) |
||||
| To merge this branch: | bzr merge lp:~nick-dedekind/unity8/inline-dash-videos | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Michał Sawicz | Needs Fixing on 2015-09-29 | ||
| Albert Astals Cid (community) | 2015-05-27 | Needs Fixing on 2015-09-18 | |
| PS Jenkins bot | continuous-integration | Needs Fixing on 2015-09-17 | |
| Jim Hodapp | Needs Fixing on 2015-06-11 | ||
|
Review via email:
|
|||
This proposal has been superseded by a proposal from 2015-12-15.
Commit Message
Added live video playback in dash previews
Description of the Change
Added live video playback in dash previews.
https:/
* Are there any related MPs required for this MP to build/function as expected? Please list.
N/A
* Did you perform an exploratory manual test run of your code change and any related functionality?
Yes
* Did you make sure that your branch does not contain spurious tags?
Yes
* If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
N/A
* If you changed the UI, has there been a design review?
Not yet.
* Did you have a look at the warnings when running tests? Can they be reduced?
N/A
| Nick Dedekind (nick-dedekind) wrote : | # |
- 1781. By Nick Dedekind on 2015-05-27
-
removed QtMultimedia req
- 1782. By Nick Dedekind on 2015-05-27
-
reverted req removal
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1782
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 1783. By Nick Dedekind on 2015-05-27
-
re-remove qtmultimedia req!
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1783
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 1784. By Nick Dedekind on 2015-06-02
-
update components version
- 1785. By Nick Dedekind on 2015-06-08
-
added copywrite. removed unused code. error handling
| Nick Dedekind (nick-dedekind) wrote : | # |
> Missing copyright headers on the new files
added
- 1786. By Nick Dedekind on 2015-06-08
-
more headers
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1784
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1786
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1786
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Jim Hodapp (jhodapp) wrote : | # |
Need to clarify with the design team about what the user experience should be when a video is larger than the size of the screen. For example, record a video using the camera-app and then preview the video in this new scope code. You'll notice it doesn't fit on the screen and is a pretty poor UX. I don't think the default behavior is desirable but should the VideoOutput size be scaled down to fit or what?
| Michael Zanetti (mzanetti) wrote : | # |
> Need to clarify with the design team about what the user experience should be
> when a video is larger than the size of the screen. For example, record a
> video using the camera-app and then preview the video in this new scope code.
> You'll notice it doesn't fit on the screen and is a pretty poor UX. I don't
> think the default behavior is desirable but should the VideoOutput size be
> scaled down to fit or what?
PreserveAspectFit, can't see any other reasonable option.
| Jim Hodapp (jhodapp) wrote : | # |
Why wouldn't stretch while shrinking the size of the VideoOutput to shrink it be an option? As long as we shrink it preserving the aspect ratio, it should still look just fine since you're not stretching it larger.
| Nick Dedekind (nick-dedekind) wrote : | # |
> Why wouldn't stretch while shrinking the size of the VideoOutput to shrink it
> be an option? As long as we shrink it preserving the aspect ratio, it should
> still look just fine since you're not stretching it larger.
Not quite sure what you mean here. What are we stretching?
We would scale the video output down to the maximum available space while keeping aspect.
| Jim Hodapp (jhodapp) wrote : | # |
> > Why wouldn't stretch while shrinking the size of the VideoOutput to shrink
> it
> > be an option? As long as we shrink it preserving the aspect ratio, it should
> > still look just fine since you're not stretching it larger.
>
> Not quite sure what you mean here. What are we stretching?
> We would scale the video output down to the maximum available space while
> keeping aspect.
That's precisely what I mean. Apparently it was a bad word day for me. ;)
- 1787. By Nick Dedekind on 2015-06-15
-
merged trunk
- 1788. By Nick Dedekind on 2015-06-19
-
better handling of sizing
- 1789. By Nick Dedekind on 2015-06-19
-
better lazy image scaling
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1789
http://
Executed test runs:
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Albert Astals Cid (aacid) wrote : | # |
Text conflict in tests/qmltests/
1 conflicts encountered.
- 1790. By Nick Dedekind on 2015-06-23
-
merged with trunk
| Nick Dedekind (nick-dedekind) wrote : | # |
> You have bad tags
fixed and merged.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1790
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1791. By Nick Dedekind on 2015-06-25
-
reverted to sdk 1.2
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1791
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Albert Astals Cid (aacid) wrote : | # |
Should we remove PreviewVideoPla
| Albert Astals Cid (aacid) wrote : | # |
There's a tst_MediaServic
make testMediaServices.
| Albert Astals Cid (aacid) wrote : | # |
When "fullscreen" one can see the "preview page" below (see Video in http://
| Albert Astals Cid (aacid) wrote : | # |
Text conflict in tests/mocks/
Text conflict in tests/mocks/
2 conflicts encountered.
| Albert Astals Cid (aacid) wrote : | # |
Change the TRANSLATORS to Translators it's what we settled on because previously we were using both and gettext only supports one.
Also run make pot_file to have an updated .pot
| Nick Dedekind (nick-dedekind) wrote : | # |
> When "fullscreen" one can see the "preview page" below (see Video in
> http://
> to me.
I followed the designs and colors specified in https:/
So I guess the answer is yes, until we're told otherwise anyway.
- 1792. By Nick Dedekind on 2015-07-08
-
merged with trunk
- 1793. By Nick Dedekind on 2015-07-08
-
moved inline video to old files
- 1794. By Nick Dedekind on 2015-07-08
-
updated translations
| Nick Dedekind (nick-dedekind) wrote : | # |
Fixed comments.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1794
http://
Executed test runs:
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Albert Astals Cid (aacid) wrote : | # |
The preview looks weird before the thumbnail arrives
http://
Maybe we need some max in the height that also takes into account the play button?
- 1795. By Nick Dedekind on 2015-07-09
-
reverted file changes
- 1796. By Nick Dedekind on 2015-07-10
-
use both inline and old video widget
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1796
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Nick Dedekind (nick-dedekind) wrote : | # |
> The preview looks weird before the thumbnail arrives
>
> http://
>
> Maybe we need some max in the height that also takes into account the play
> button?
Should be fixed now. Initial screenshot height is playButton height + gu(2) margin.
| Albert Astals Cid (aacid) wrote : | # |
going back and forth from to a preview of a video that i recorded with the phone ends up in crashes in media-hub and that freezes the dash.
Can we make the media-hub crashing to freeze the dash? Are we waiting for something in a sync manner that should be async?
| Jim Hodapp (jhodapp) wrote : | # |
Albert: Can you get a back trace of all of the threads plus a threads info
listing and the media-hub.log for the crash?
On Tue, Jul 14, 2015 at 7:23 AM, Albert Astals Cid <
<email address hidden>> wrote:
> Review: Needs Fixing
>
> going back and forth from to a preview of a video that i recorded with the
> phone ends up in crashes in media-hub and that freezes the dash.
>
> Can we make the media-hub crashing to freeze the dash? Are we waiting for
> something in a sync manner that should be async?
> --
>
> https:/
> You are reviewing the proposed merge of
> lp:~nick-dedekind/unity8/inline-dash-videos into lp:unity8.
>
| Albert Astals Cid (aacid) wrote : | # |
> Albert: Can you get a back trace of all of the threads plus a threads info
> listing and the media-hub.log for the crash?
Jim: here it comes
bt of threads: http://
media-hub.log: http://
| Jim Hodapp (jhodapp) wrote : | # |
That looks good, thanks Albert
On Tue, Jul 14, 2015 at 11:12 AM, Albert Astals Cid <
<email address hidden>> wrote:
> > Albert: Can you get a back trace of all of the threads plus a threads
> info
> > listing and the media-hub.log for the crash?
>
> Jim: here it comes
>
> bt of threads: http://
> media-hub.log: http://
> --
>
> https:/
> You are reviewing the proposed merge of
> lp:~nick-dedekind/unity8/inline-dash-videos into lp:unity8.
>
| Albert Astals Cid (aacid) wrote : | # |
Text conflict in tests/qmltests/
1 conflicts encountered.
| Albert Astals Cid (aacid) wrote : | # |
You can probably remove
item.
Once we merge
https:/
| Michael Zanetti (mzanetti) wrote : | # |
mardy's branch is merged.
| Albert Astals Cid (aacid) wrote : | # |
Please propose against overlay
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1796
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Albert Astals Cid (aacid) wrote : | # |
Ignore the proposing to overlay, we don't use it anymore, you still have conflicts though
Text conflict in qml/Dash/
Text conflict in tests/mocks/
Text conflict in tests/qmltests/
3 conflicts encountered.
| Michał Sawicz (saviq) wrote : | # |
Text conflict in qml/Dash/
Text conflict in tests/mocks/
Text conflict in tests/qmltests/
3 conflicts encountered.

Commented out sections in VideoPlayer is waiting on a design review (may need the ShaderEffect), and the in PreviewInlineVi deo.qml is for sharing via content hub.
Will be removed before merging.