Bug 335356 - Changes for RAP/RCP single-sourced version of Memory Analyzer
Summary: Changes for RAP/RCP single-sourced version of Memory Analyzer
Status: CLOSED MOVED
Alias: None
Product: MAT
Classification: Tools
Component: GUI (show other bugs)
Version: 1.1   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL: https://dev.eclipse.org/ipzilla/show_...
Whiteboard:
Keywords:
Depends on:
Blocks: 262603
  Show dependency tree
 
Reported: 2011-01-25 13:04 EST by Andrew Johnson CLA
Modified: 2024-05-08 14:42 EDT (History)
1 user (show)

See Also:


Attachments
Patch to existing MAT projects to enable RAP (107.89 KB, patch)
2011-01-25 13:10 EST, Andrew Johnson CLA
no flags Details | Diff
Additional plugin projects for MAT for web enablement using RAP (21.19 KB, application/zip)
2011-01-25 13:13 EST, Andrew Johnson CLA
no flags Details
Modify eclipse help system for use in RAP environment (200.51 KB, patch)
2011-01-25 13:16 EST, Andrew Johnson CLA
no flags Details | Diff
RAP compatibility, forwarding UI calls from MAT to RAP (64.07 KB, application/zip)
2011-01-25 13:17 EST, Andrew Johnson CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Johnson CLA 2011-01-25 13:04:40 EST
CQ 4723 approved the changes from bug 262603 comment 1 for use in the Memory Analyzer project.

The approved code will be attached to this bug.
Comment 1 Andrew Johnson CLA 2011-01-25 13:10:07 EST
Created attachment 187550 [details]
Patch to existing MAT projects to enable RAP

Approved by CQ 4723
Comment 2 Andrew Johnson CLA 2011-01-25 13:13:47 EST
Created attachment 187552 [details]
Additional plugin projects for MAT for web enablement using RAP

Approved by CQ 4723

Hi Andrew:

I've been able to complete full review as part of the triage stage.

A couple of small changes that should be made before the content is checked in:

1.  ActionFactoryFacadeImpl.java contains the following 
 "About", "(c) Copyright Eclipse contributors and others 2000, 2009.  All
rights reserved.")

Copyright should not be assigned to Eclipse contributors (please note this
information is not contained in the header but included quite a distance down
in the file)

2. ...

Otherwise, this CQ is approved!

Cheers,
Sharon
Comment 3 Andrew Johnson CLA 2011-01-25 13:16:15 EST
Created attachment 187553 [details]
Modify eclipse help system for use in RAP environment

Approved by CQ 4723

Hi Andrew:

I've been able to complete full review as part of the triage stage.

A couple of small changes that should be made before the content is checked in:

1.  ...

2.  The patch contains the word 'hack', you may wish to adjust:-)

Otherwise, this CQ is approved!

Cheers,
Sharon
Comment 4 Andrew Johnson CLA 2011-01-25 13:17:30 EST
Created attachment 187554 [details]
RAP compatibility, forwarding UI calls from MAT to RAP

Approved by CQ 4723
Comment 5 Eclipse Webmaster CLA 2024-05-08 14:42:33 EDT
This issue has been migrated to https://github.com/eclipse-mat/org.eclipse.mat/issues/16.