Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: installer/src/msi/adblockplusie.wxs

Issue 4889997257736192: Issue 1821 - Add product support URL to installer (Closed)
Left Patch Set: Created March 13, 2015, 1:15 p.m.
Right Patch Set: Removing comment Created March 20, 2015, 4:43 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « no previous file | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 <?xml version="1.0" encoding="UTF-8"?> 1 <?xml version="1.0" encoding="UTF-8"?>
2 <!-- 2 <!--
3 Compilation Note: 3 Compilation Note:
4 You may see linking errors that look like these: 4 You may see linking errors that look like these:
5 <quote>LGHT0217 : Error executing ICE action 'ICExx' [...] The error code is 2738.</quote> 5 <quote>LGHT0217 : Error executing ICE action 'ICExx' [...] The error code is 2738.</quote>
6 The documentation for this error code talks about custom actions, 6 The documentation for this error code talks about custom actions,
7 even though the CustomAction keyword does not appear in this source. 7 even though the CustomAction keyword does not appear in this source.
8 Certain WiX elements are implemented by MSI custom actions, 8 Certain WiX elements are implemented by MSI custom actions,
9 which means this source implicitly declares some insofar as the Windows Inst aller sees it. 9 which means this source implicitly declares some insofar as the Windows Inst aller sees it.
10 These errors are benign and can be avoided by some fiddling with the registry on the development machine. 10 These errors are benign and can be avoided by some fiddling with the registry on the development machine.
(...skipping 130 matching lines...) Expand 10 before | Expand all | Expand 10 after
141 141
142 <!-- 142 <!--
143 User interface 143 User interface
144 --> 144 -->
145 <UIRef Id="custom_WixUI_InstallDir"/> 145 <UIRef Id="custom_WixUI_InstallDir"/>
146 <UIRef Id="WixUI_ErrorProgressText"/> 146 <UIRef Id="WixUI_ErrorProgressText"/>
147 <Property Id="WIXUI_INSTALLDIR" Value="INSTALLFOLDER"/> 147 <Property Id="WIXUI_INSTALLDIR" Value="INSTALLFOLDER"/>
148 148
149 <!-- 149 <!--
150 Icon for Add/Remove Programs 150 Icon for Add/Remove Programs
151
152 It currently uses the blocking icon.
153 When #1538 is ready, we may have a separate icon file for this.
154 --> 151 -->
Eric 2015/03/13 16:37:11 Can we kill this comment now?
155 <Icon Id="program_icon" SourceFile="$(var.plugin_dir)\src\res\blocking.ico"/ > 152 <Icon Id="program_icon" SourceFile="$(var.plugin_dir)\src\res\blocking.ico"/ >
156 <Property Id="ARPPRODUCTICON" Value="program_icon"/> 153 <Property Id="ARPPRODUCTICON" Value="program_icon"/>
157 <Property Id="ARPURLINFOABOUT" Value="https://adblockplus.org/en/about"/> 154 <Property Id="ARPURLINFOABOUT" Value="https://adblockplus.org/en/about"/>
158 <Property Id="ARPURLUPDATEINFO" Value="https://adblockplus.org/category/adbl ock-plus-ie/"/> 155 <Property Id="ARPURLUPDATEINFO" Value="https://adblockplus.org/category/adbl ock-plus-ie/"/>
159 <Property Id="ARPHELPLINK" Value="https://adblockplus.org/en/bugs"/> 156 <Property Id="ARPHELPLINK" Value="https://adblockplus.org/en/bugs"/>
Eric 2015/03/13 16:37:11 We could usefully add ARPSIZE here as well. For wh
Oleksandr 2015/03/20 04:44:33 I'd rather not. Currently the correct size is disp
160
161
162 157
163 <!-- 158 <!--
164 Define and schedule a "Custom Action Type 1" to ensure that IE is closed b efore installation. 159 Define and schedule a "Custom Action Type 1" to ensure that IE is closed b efore installation.
165 Because WiX is closely tied to the MSI database format, there are multiple tags required to do this. 160 Because WiX is closely tied to the MSI database format, there are multiple tags required to do this.
166 The "CustomAction" tag defines the tag itself, but doesn't sequence it; 161 The "CustomAction" tag defines the tag itself, but doesn't sequence it;
167 it defines an entry in the "CustomAction" table. 162 it defines an entry in the "CustomAction" table.
168 The "Binary" tag describe an asset that's incorporated into the MSI but th at is not an installed component. 163 The "Binary" tag describe an asset that's incorporated into the MSI but th at is not an installed component.
169 The "Custom" tag defines when the action executes by defining an entry in one of the sequence tables. 164 The "Custom" tag defines when the action executes by defining an entry in one of the sequence tables.
170 165
171 Attribute "Return" sets the "Custom Action Return Processing Options" to z ero, 166 Attribute "Return" sets the "Custom Action Return Processing Options" to z ero,
(...skipping 656 matching lines...) Expand 10 before | Expand all | Expand 10 after
828 <Component Id="ini_settings.ini" Guid="dc30c3b0-d713-11e2-8b8b-0800200c9 a66"> 823 <Component Id="ini_settings.ini" Guid="dc30c3b0-d713-11e2-8b8b-0800200c9 a66">
829 <File Name="settings.ini"/> 824 <File Name="settings.ini"/>
830 </Component> 825 </Component>
831 </Directory> 826 </Directory>
832 </DirectoryRef> 827 </DirectoryRef>
833 828
834 <?include ../installer-lib/custom-i18n.wxi ?> 829 <?include ../installer-lib/custom-i18n.wxi ?>
835 <?include ../custom-action/close_ie.wxi ?> 830 <?include ../custom-action/close_ie.wxi ?>
836 </Product> 831 </Product>
837 </Wix> 832 </Wix>
LEFTRIGHT
« no previous file | no next file » | Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Toggle Comments ('s')

Powered by Google App Engine
This is Rietveld