31f1b0aa5308eebc39635a76db6016e0598e33ad: Bug 1299753 Part 2 - Create block formatting context for DetailsFrame if needed. r=bz
Ting-Yu Lin <tlin@mozilla.com> - Wed, 07 Sep 2016 14:45:42 +0800 - rev 313295
Push 32161 by tlin@mozilla.com at Fri, 09 Sep 2016 07:06:30 +0000
Bug 1299753 Part 2 - Create block formatting context for DetailsFrame if needed. r=bz nsCSSFrameConstructor::ConstructNonScrollableBlock() has logic to determine whether to create a block formatting context for a block frame. I refactor the function to make it reusable by nsCSSFrameConstructor::ConstructDetailsFrame(). Also, make NS_NewBlockFrame() accept two arguments as other frame factory functions so that it could be pointed by BlockFrameCreationFunc. NS_NewBlockFormattingContext is changed accordingly. The construction for a scrollable DetailsFrame will be further revised in Part 3. MozReview-Commit-ID: 8TwG9YMyGva
8d8754d431f311605926dbb8414337aa39ba059b: Bug 1299753 Part 1 - Use NS_NewBlockFormattingContext() in ConstructFieldSetFrame(). r=bz
Ting-Yu Lin <tlin@mozilla.com> - Mon, 05 Sep 2016 15:36:53 +0800 - rev 313294
Push 32161 by tlin@mozilla.com at Fri, 09 Sep 2016 07:06:30 +0000
Bug 1299753 Part 1 - Use NS_NewBlockFormattingContext() in ConstructFieldSetFrame(). r=bz MozReview-Commit-ID: E6lvCRIDROY
5947f7b762ef39c1bb33916cc20e7c849afdf51c: Bug 1301514 - Destroy browser API frame scripts during swap. r=kanru
J. Ryan Stinnett <jryans@gmail.com> - Thu, 08 Sep 2016 16:00:12 -0500 - rev 313293
Push 32160 by jryans@gmail.com at Fri, 09 Sep 2016 05:32:41 +0000
Bug 1301514 - Destroy browser API frame scripts during swap. r=kanru When swapping content from <iframe mozbrowser> to <xul:browser>, we now stop the frame scripts that implement the content side of the browser API since they are no longer needed and can cause issues if they remain active. MozReview-Commit-ID: JrecxA4MI93
30a7de6065d713e29926e60a80b5f8cc568b0680: Bug 1301514 - Clarify ownership of BrowserElementIsReady. r=kanru
J. Ryan Stinnett <jryans@gmail.com> - Wed, 07 Sep 2016 17:45:13 -0500 - rev 313292
Push 32160 by jryans@gmail.com at Fri, 09 Sep 2016 05:32:41 +0000
Bug 1301514 - Clarify ownership of BrowserElementIsReady. r=kanru It wasn't immediately obvious to me that BrowserElementIsReady was correctly guarding against re-running the browser element scripts in a frame. After more testing, it was working, but I've added some debug lines for future clarity. No functionality changes in this patch. MozReview-Commit-ID: CW4o2TsGKmj
52a630b9ed23a9ca199837644a616dfa8407339a: Bug 1290467 - part10 : update audio focus related control operations. r=sebastian
Alastor Wu <alwu@mozilla.com> - Fri, 09 Sep 2016 09:51:12 +0800 - rev 313291
Push 32159 by alwu@mozilla.com at Fri, 09 Sep 2016 05:07:37 +0000
Bug 1290467 - part10 : update audio focus related control operations. r=sebastian Except controlling audio focus from gecko, the MediaControlService can also decide whether needs to request or abandon audio focus. MozReview-Commit-ID: G3iSYwd24JZ
0ccb80fc769b96d1f342d6fcc5e6a748d1a186f4: Bug 1290467 - part9 : remove 'ACTION_REMOVE_CONTROL'. r=sebastian
Alastor Wu <alwu@mozilla.com> - Fri, 09 Sep 2016 09:51:10 +0800 - rev 313290
Push 32159 by alwu@mozilla.com at Fri, 09 Sep 2016 05:07:37 +0000
Bug 1290467 - part9 : remove 'ACTION_REMOVE_CONTROL'. r=sebastian Remove 'ACTION_REMOVE_CONTROL' because it's as same as 'ACTION_STOP'. MozReview-Commit-ID: 6KOj8srEuJA
9c459f432b365a3c209f150df5864d3299a91e1f: Bug 1290467 - part8 : rename 'ACTION_PLAY' to 'ACTION_RESUME'. r=sebastian
Alastor Wu <alwu@mozilla.com> - Fri, 09 Sep 2016 09:51:08 +0800 - rev 313289
Push 32159 by alwu@mozilla.com at Fri, 09 Sep 2016 05:07:37 +0000
Bug 1290467 - part8 : rename 'ACTION_PLAY' to 'ACTION_RESUME'. r=sebastian 'ACTION_RESUME' should be more suit for its operation. MozReview-Commit-ID: 4FRHaydVKu5
789d97d6ca60016c83b5b1b5dade71f681c9bfb7: Bug 1290467 - part7 : enable audio competing for non-audible media. r=baku
Alastor Wu <alwu@mozilla.com> - Fri, 09 Sep 2016 09:50:40 +0800 - rev 313288
Push 32159 by alwu@mozilla.com at Fri, 09 Sep 2016 05:07:37 +0000
Bug 1290467 - part7 : enable audio competing for non-audible media. r=baku In general, the audio competing should only be for audible media and it helps user can focus on one media at the same time. However, we hope to treat all media as the same in the mobile device. First reason is we have media control on fennec and we just want to control one media at once time. Second reason is to reduce the bandwidth, avoiding to play any non-audible media in background which user doesn't notice about. MozReview-Commit-ID: yB3181cmVE
7eb048d423d17b3b0a3360897b5f71ed602d670e: Bug 1290467 - part6 : remove redudant space. r=sebastian
Alastor Wu <alwu@mozilla.com> - Fri, 09 Sep 2016 09:50:32 +0800 - rev 313287
Push 32159 by alwu@mozilla.com at Fri, 09 Sep 2016 05:07:37 +0000
Bug 1290467 - part6 : remove redudant space. r=sebastian remove one redudant space. MozReview-Commit-ID: 1PdKL1EALAq
e9512bfae27b0ddabf161a00f6113d1a337e4917: Bug 1290467 - part5 : change service's life time. r=sebastian
Alastor Wu <alwu@mozilla.com> - Fri, 09 Sep 2016 09:50:30 +0800 - rev 313286
Push 32159 by alwu@mozilla.com at Fri, 09 Sep 2016 05:07:37 +0000
Bug 1290467 - part5 : change service's life time. r=sebastian Now the life time of the MediaControlService would be as same as the Fennec app. To make code flow more easily, requesting/abandoning the audio focus wouldn't affect the media control. We would mainly communicate with the media control via TabEvents. MozReview-Commit-ID: KT59bII0HuN
f1f65fd576fc78763b7f2221200c62151aae7f7b: Bug 1290467 - part4 : create helper function. r=sebastian
Alastor Wu <alwu@mozilla.com> - Fri, 09 Sep 2016 09:50:28 +0800 - rev 313285
Push 32159 by alwu@mozilla.com at Fri, 09 Sep 2016 05:07:37 +0000
Bug 1290467 - part4 : create helper function. r=sebastian wrap some code into initialize() and shutdown(). MozReview-Commit-ID: AiyABlyDEME
fdc63760ddc8fca6b9e986b86ae4ac4e1da0b9a7: Bug 1290467 - part3 : notify MediaControlService with event 'MEDIA_PLAYING_CHANGE'. r=sebastian
Alastor Wu <alwu@mozilla.com> - Fri, 09 Sep 2016 09:50:25 +0800 - rev 313284
Push 32159 by alwu@mozilla.com at Fri, 09 Sep 2016 05:07:37 +0000
Bug 1290467 - part3 : notify MediaControlService with event 'MEDIA_PLAYING_CHANGE'. r=sebastian The 'MEDIA_PLAYING_CHANGE' is used for controling media control interface and the 'AUDIO_PLAYING_CHANGE' is used for showing the tab sound indicator. MozReview-Commit-ID: 8hZjC77Ju71
ef1a345f206baf53604b4fff52e46e552623a3bd: Bug 1290467 - part2 : request audio focus for any media instead of audible one. r=snorp
Alastor Wu <alwu@mozilla.com> - Fri, 09 Sep 2016 09:49:25 +0800 - rev 313283
Push 32159 by alwu@mozilla.com at Fri, 09 Sep 2016 05:07:37 +0000
Bug 1290467 - part2 : request audio focus for any media instead of audible one. r=snorp Previous design is only to request audio focus for audible media, but now we also request focus for non-audible media. It's simple that the app should own the focus when users start watching media. MozReview-Commit-ID: 3eJP26h4kh7
c3210fd31140cd5bbe2e35c3439ba2141b8f8a6d: Bug 1290467 - part1 : dispatch 'media-playback' event. r=baku
Alastor Wu <alwu@mozilla.com> - Fri, 09 Sep 2016 09:49:22 +0800 - rev 313282
Push 32159 by alwu@mozilla.com at Fri, 09 Sep 2016 05:07:37 +0000
Bug 1290467 - part1 : dispatch 'media-playback' event. r=baku Use 'media-playback' event to control the media control interface on Fennec. MozReview-Commit-ID: D8SU96RrkbQ
12d5778a37a6e00af12e7272997cebf101104eea: Bug 1294003 - Ensure we destroy DXVA2Manager on main thread if DXVA init fails. r=mattwoodrow
Chris Pearce <cpearce@mozilla.com> - Fri, 09 Sep 2016 16:12:50 +1200 - rev 313281
Push 32158 by cpearce@mozilla.com at Fri, 09 Sep 2016 04:47:14 +0000
Bug 1294003 - Ensure we destroy DXVA2Manager on main thread if DXVA init fails. r=mattwoodrow If initializing DXVA fails, we end up destroying the DXVA2Manager on the decode task queue. The DXVA2Manager asserts that it's destroyed on the main thread, so we should dispatch a task to destroy it on the appropriate thread instead of destorying it on the wrong one. MozReview-Commit-ID: 2pbeMOm74et
93c6fdc10d2a0507d7dbde688f4ab3144f95ca42: Bug 1288350 - Temporary reftest list for Stylo vs Gecko test r=heycam,manishearth
Shing Lyu <shing.lyu@gmail.com> - Mon, 05 Sep 2016 11:45:39 +0800 - rev 313280
Push 32157 by cmccormack@mozilla.com at Fri, 09 Sep 2016 04:16:35 +0000
Bug 1288350 - Temporary reftest list for Stylo vs Gecko test r=heycam,manishearth
da6188fe9f7d03f821b594d3c67cac6c4b248295: Bug 1301561 - Change media.decoder.heuristic.dormant.timeout to 10000 (ms) - r=jwwang
Gerald Squelart <gsquelart@mozilla.com> - Fri, 09 Sep 2016 11:00:21 +1000 - rev 313279
Push 32156 by gsquelart@mozilla.com at Fri, 09 Sep 2016 02:19:14 +0000
Bug 1301561 - Change media.decoder.heuristic.dormant.timeout to 10000 (ms) - r=jwwang MozReview-Commit-ID: BibXsBpjIQe
76ded056a9dec866e29ba13b4248e4e7e16547f5: Bug 1298979 - move tabs.sendMessage/connect to child process r=billm
Rob Wu <rob@robwu.nl> - Sun, 28 Aug 2016 20:07:46 -0700 - rev 313278
Push 32155 by ryanvm@gmail.com at Fri, 09 Sep 2016 01:49:50 +0000
Bug 1298979 - move tabs.sendMessage/connect to child process r=billm - Use the frame's message manager to direct messages via the ProxyMessenger to the right tab instead of directly to the tab. - Put the implementation in a separate file that is only loaded in child processes (in the future). - Explicitly list all addon-process specific files in a new category instead of reusing the content one. MozReview-Commit-ID: 8oIMx9ol7Tl
2fa53669115d638c51a641f579344d3a107af767: Bug 1298979 - Add test to verify that sending a message and unloading works r=billm
Rob Wu <rob@robwu.nl> - Sun, 28 Aug 2016 16:26:52 -0700 - rev 313277
Push 32155 by ryanvm@gmail.com at Fri, 09 Sep 2016 01:49:50 +0000
Bug 1298979 - Add test to verify that sending a message and unloading works r=billm Checks what happens before closing a window or removing a frame: - Tests that sendMessage/connect is received by the extension. - Tests that any responses from the extension is not received by the sending script (of the closing context). MozReview-Commit-ID: 9VwCpRmaZOO
f9f94cf30f5342dfa4acb3ec9c1be60f5874d673: Bug 1298979 - Check whether ProxyContext exists before using it r=billm
Rob Wu <rob@robwu.nl> - Thu, 25 Aug 2016 19:36:30 -0700 - rev 313276
Push 32155 by ryanvm@gmail.com at Fri, 09 Sep 2016 01:49:50 +0000
Bug 1298979 - Check whether ProxyContext exists before using it r=billm Due to asynchronicity or malice we can receive messages for unknown ProxyContexts. Immediately reject such messages. (this addresses https://bugzil.la/1288902#c3) MozReview-Commit-ID: GEgkZC8CUEG
(0) -300000 -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip