Closed
Bug 923596
Opened 11 years ago
Closed 10 years ago
Integrate Shumway code into Metro Firefox (disabled by default)
Categories
(Firefox for Metro Graveyard :: Components, enhancement)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 29
People
(Reporter: mbrubeck, Assigned: mbrubeck)
References
Details
Attachments
(1 file)
3.94 KB,
patch
|
gps
:
review+
jimm
:
review+
|
Details | Diff | Splinter Review |
This adds support for the Shumway extension (JavaScript/HTML player for SWF) to Metro Firefox -- disabled by default for now, as it is in desktop Firefox.
Attachment #813680 -
Flags: review?(jmathies)
Attachment #813680 -
Flags: review?(gps)
Attachment #813680 -
Flags: feedback?(ydelendik)
Comment 1•11 years ago
|
||
Comment on attachment 813680 [details] [diff] [review] patch Review of attachment 813680 [details] [diff] [review]: ----------------------------------------------------------------- r+ on metro specific changes. ::: browser/metro/base/content/browser-ui.js @@ +111,5 @@ > PageThumbs.init(); > NewTabUtils.init(); > SettingsCharm.init(); > NavButtonSlider.init(); > + ShumwayUtils.init(); do we need to call this if it's not enabled via prefs?
Attachment #813680 -
Flags: review?(jmathies) → review+
Comment 2•11 years ago
|
||
Touch events simulation is not implemented atm, see https://github.com/mozilla/shumway/issues/767 . So it might be a problem, similar to the bug 859579
Assignee | ||
Comment 3•11 years ago
|
||
(In reply to Jim Mathies [:jimm] from comment #1) > > + ShumwayUtils.init(); > > do we need to call this if it's not enabled via prefs? Yes -- calling it adds a pref observer, so that toggling the pref via about:config will enable the add-on. (In reply to Yury Delendik (:yury) from comment #2) > Touch events simulation is not implemented atm, see > https://github.com/mozilla/shumway/issues/767 . So it might be a problem, > similar to the bug 859579 Yes, I think we should block on good touch support before enabling Shumway by default. But many Windows 8 users have mouse or trackpad instead of (or in addition to) touch, so it will still be easy for most users to test Shumway in Metro.
Comment 4•11 years ago
|
||
Comment on attachment 813680 [details] [diff] [review] patch Review of attachment 813680 [details] [diff] [review]: ----------------------------------------------------------------- Reluctant r+ because of poor makeisms. But it's no worse than what's already in this Makefile.in
Attachment #813680 -
Flags: review?(gps) → review+
Assignee | ||
Comment 5•11 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/00a62ede0d4d
Assignee | ||
Updated•11 years ago
|
Attachment #813680 -
Flags: feedback?(ydelendik)
Assignee | ||
Comment 6•11 years ago
|
||
Backed out along with bug 925796 because one of them caused Metro to crash/hang: https://hg.mozilla.org/integration/fx-team/rev/5ec2fca2c0b0
Assignee | ||
Comment 7•10 years ago
|
||
Pushed to Try to find out whether this still breaks our test automation: https://tbpl.mozilla.org/?tree=Try&rev=c17feef46411
Assignee | ||
Comment 8•10 years ago
|
||
Fixed the failure by adding a missing change to the package manifest: https://tbpl.mozilla.org/?tree=Try&rev=faed35a5ca67 Updated to latest trunk and landed on fx-team: https://hg.mozilla.org/integration/fx-team/rev/6370458acfd9
Assignee | ||
Comment 9•10 years ago
|
||
This was merged a while ago but wasn't resolved due to oversight. https://hg.mozilla.org/mozilla-central/rev/6370458acfd9
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 29
Assignee | ||
Updated•10 years ago
|
Blocks: metro-shumway
Assignee | ||
Updated•10 years ago
|
Updated•10 years ago
|
OS: Windows 8 Metro → Windows 8.1
You need to log in
before you can comment on or make changes to this bug.
Description
•