| Summary: | Customized External Browser is not used | ||||||
|---|---|---|---|---|---|---|---|
| Product: | z_Archived | Reporter: | Roy Ganor <ganoro> | ||||
| Component: | ATF | Assignee: | Inbox <atf.core-inbox> | ||||
| Status: | NEW --- | QA Contact: | |||||
| Severity: | normal | ||||||
| Priority: | P3 | CC: | jacek.pospychala | ||||
| Version: | unspecified | ||||||
| Target Milestone: | --- | ||||||
| Hardware: | PC | ||||||
| OS: | Windows 7 | ||||||
| Whiteboard: | |||||||
| Attachments: |
|
||||||
|
Description
Roy Ganor
Created attachment 185626 [details]
patch
I don't like the iexplore path editing. Can't we simply use "iexplore.exe"? It should always be on path.
I don't know what is returned by BrowserManager.getCurrentWebBrowser(). It will return browser selected by user, yes?
I also don't understand this change:
+ } else {
+ if ((style & IWorkbenchBrowserSupport.AS_VIEW) != 0)
+ webBrowser = new InternalBrowserViewInstance(browserId, style,
+ name, tooltip);
+ else
+ webBrowser = new InternalBrowserEditorInstance(browserId,
+ style, name, tooltip);
+ }
does it mean that in some circumstances, the Platform's internal browser will be used?
|