Bug 28058 - Xam.Mac QTRecorder throws exception on changing/Selecting 'Video Preview' under 'Recording Options'.
Summary: Xam.Mac QTRecorder throws exception on changing/Selecting 'Video Preview' und...
Status: VERIFIED FIXED
Alias: None
Product: Xamarin.Mac
Classification: Desktop
Component: Samples ()
Version: Master
Hardware: PC Mac OS
: High major
Target Milestone: 2.0.x
Assignee: Chris Hamons
URL:
Depends on:
Blocks:
 
Reported: 2015-03-16 10:02 UTC by Rajneesh Kumar
Modified: 2015-03-27 07:18 UTC (History)
4 users (show)

Tags:
Is this bug a regression?: ---
Last known good build:

Notice (2018-05-24): bugzilla.xamarin.com is now in read-only mode.

Please join us on Visual Studio Developer Community and in the Xamarin and Mono organizations on GitHub to continue tracking issues. Bugzilla will remain available for reference in read-only mode. We will continue to work on open Bugzilla bugs, copy them to the new locations as needed for follow-up, and add the new items under Related Links.

Our sincere thanks to everyone who has contributed on this bug tracker over the years. Thanks also for your understanding as we make these adjustments and improvements for the future.


Please create a new report on Developer Community or GitHub with your current version information, steps to reproduce, and relevant error messages or log files if you are hitting an issue that looks similar to this resolved bug and you do not yet see a matching new report.

Related Links:
Status:
VERIFIED FIXED

Comment 1 Chris Hamons 2015-03-16 16:59:17 UTC
It is throwing because:

			CIFilter f = new CIGloom ();
			Console.WriteLine (f.RespondsToSelector (new Selector ("inputImage")));
			Console.WriteLine (f.RespondsToSelector (new Selector ("outputImage")));

is returning False / True and this code throws.

				if (!SupportsInputImage)
					throw new ArgumentException ("inputImage is not supported by this filter");
				SetInputImage (value);

Question is why is this returning false. Did this ever work?
Comment 2 Chris Hamons 2015-03-16 16:59:54 UTC
How did the break from  1.12.0.4 to 1.12.0.14?
Comment 3 Chris Hamons 2015-03-16 17:01:19 UTC
Probably 3dd6cbe4851a0e909f771a8afd7dc9043632cdfe broke it. Checking tomorrow.
Comment 4 Rustam Zaitov 2015-03-16 17:02:31 UTC
Actually CIFilter has the typed properties for input and output images, so we should use them instead of selectors
Comment 5 Chris Hamons 2015-03-16 17:03:56 UTC
The code under the hood is still broken introduced in 3dd6cbe4851a0e909f771a8afd7dc9043632cdfe.
Comment 6 Chris Hamons 2015-03-17 12:18:26 UTC
This was completely broken due to that change. Great catch!

I've added an API test to hopefully prevent it from happening in the future.
Comment 7 Chris Hamons 2015-03-25 16:41:56 UTC
Fixed in 8b86e15ba8846707b2e4b700e49a52e18ad96b54 / master.
Comment 8 Rajneesh Kumar 2015-03-27 07:18:07 UTC
I have checked this issue with the following builds:

=== Xamarin Studio ===
Version 5.9 (build 310)
Installation UUID: 011d70a5-dede-428b-ab04-ef451c2e539d
Runtime:
Mono 4.0.0 ((detached/87c62a1)
GTK+ 2.24.23 (Raleigh theme)
Package version: 400000116
Apple Developer Tools Xcode 6.2 (6776)
Build 6C131e
Xamarin.Mac Version: 2.0.0.174 (Business Edition)
=== Build Information ===
Release ID: 509000310
Git revision: 5e63e3a7face15bd906106306fb4565eaf503995
Build date: 2015-03-26 12:30:38-04
Xamarin addins: 14cabb8b8f6b94fb7aac79016d64f4b9db4d0091
Operating System Mac OS X 10.9.5
Darwin MacMini.local 13.4.0 Darwin Kernel Version 13.4.0
Sun Aug 17 19:50:11 PDT 2014
root:xnu-2422.115.4~1/RELEASE_X86_64 x86_64

I observed that this issue is working fine. Now, on Changing/Selecting Video Preview option user does't get exception. Please refer the screencast: http://www.screencast.com/t/xUNpPBxj

This issue has been fixed, hence I am closing this issue.
Thanks!