Bug 428845

Summary: ColorDialog: Begin implementation
Product: [Mono] UI Automation Reporter: Brad Taylor <btaylor1>
Component: Winforms - UIAAssignee: E-mail List <mono-a11y-bugs>
Status: CONFIRMED --- QA Contact: E-mail List <mono-a11y-qa>
Severity: Enhancement    
Priority: P4 - Low    
Version: Unspecified   
Target Milestone: Release 1.1   
Hardware: Other   
OS: Other   
Whiteboard:
Found By: --- Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---
Bug Depends on:    
Bug Blocks: 428474    
Attachments: Patch Uploaded.
Patch Uploaded.
Fixed Invoke action.
unit test

Description Brad Taylor 2008-09-22 21:54:13 UTC
Begin initial work on ColorDialog including navigation for automation tree, and develop initial unit tests.
Comment 1 Andres Aragoneses 2008-09-25 18:07:04 UTC
Mass change of QA contact. Sorry for bugspam. Update your watch-lists.
Comment 2 Brad Taylor 2008-09-25 22:12:37 UTC
Moving all of these to P4 to show that they have been triaged.  Will be reprioritizing these properly tomorrow.
Comment 3 Neville Gao 2008-12-29 03:26:38 UTC
Created attachment 262582 [details]
Patch Uploaded.

BaseColorControl still has SmallColorControl, which is color square.
Comment 4 Neville Gao 2009-01-04 10:52:12 UTC
Created attachment 263105 [details]
Patch Uploaded.
Comment 5 Mario Carrion 2009-01-05 12:05:03 UTC
(In reply to comment #4 from Neville Gao)
> Created an attachment (id=263105) [details]
> Patch Uploaded.
OK to commit.
Comment 6 Neville Gao 2009-01-07 06:36:11 UTC
Committed in r122613.
Comment 7 Neville Gao 2009-01-07 10:02:25 UTC
Created attachment 263526 [details]
Fixed Invoke action.
Comment 8 Mario Carrion 2009-01-07 14:45:09 UTC
(In reply to comment #7 from Neville Gao)
> Created an attachment (id=263526) [details]
> Fixed Invoke action.
OK to commit
Comment 9 Neville Gao 2009-01-07 15:18:18 UTC
Committed in r122661.
Comment 10 Neville Gao 2009-01-12 11:49:44 UTC
Missing unit test.

How do we get BaseColorControl using Navigate, we couldn't get the exact one using Navigate.
Comment 11 Mario Carrion 2009-01-12 14:56:46 UTC
Not sure, did you try showing your color dialog and then looping BaseTest.bridge.StructureChangedEvents until getting the WindowProvider with Parent != null?
Comment 12 Neville Gao 2009-02-05 07:18:47 UTC
I was trying to get ColorDialog using GetProviderFromControl(), and then loop it's children. If the child has children, it must be BaseColorControl. But GetProviderFromControl() can't accept a component-type argument. Is there a way to test dialog based control?
Comment 13 Mario Carrion 2009-02-05 13:48:04 UTC
Yes, our tests need love (generics maybe?) In the meantime use GetProvider to return ProviderFactory.GetProvider, that should work.
Comment 14 Andres Aragoneses 2009-02-11 17:31:35 UTC
Neville: what's the status on this?
Comment 15 Neville Gao 2009-02-12 09:49:49 UTC
Created attachment 272206 [details]
unit test
Comment 16 Neville Gao 2009-02-12 10:48:57 UTC
Now, we can get BaseColorControl using ShowDialog() in another thread, but have a problem getting internal property with Helper, as Test class is not "friends" of Providers class.
Comment 17 Andres Aragoneses 2009-02-12 16:18:01 UTC
Mario, could you review? Thanks.
Comment 18 Mario Carrion 2009-02-12 17:34:30 UTC
(In reply to comment #15)
> Created an attachment (id=272206) [details]
> unit test
The patch looks OK.


(In reply to comment #16)
> Now, we can get BaseColorControl using ShowDialog() in another thread, but have
> a problem getting internal property with Helper, as Test class is not "friends"
> of Providers class.
Right, UIAutomationWinforms is friend of MWF, and UIAutomationWinformsTest is friend of UIAutomationWinforms, but friendship is not transitive nor reciprocal.

However, did you try navigating your BaseColorControl and searching for your Helper provider? that should work.
Comment 19 Neville Gao 2009-02-13 10:33:43 UTC
Committed in r126801.


(In reply to comment #18)
> However, did you try navigating your BaseColorControl and searching for your
> Helper provider? that should work.
Oh, seems BaseColorControl only navigates to SmallColorControls.
Comment 20 Brad Taylor 2009-03-09 13:42:40 UTC
Moving enhancement requests (e.g.: new features pending completion) to the 1.1 milestone.