Add IWindowsInstallerBackendBinder.FullyResolved plus documentation

2 files changed, 39 insertions, 5 deletions

diff --git a/src/WixToolset.Extensibility/BaseWindowsInstallerBackendBinderExtension.cs b/src/WixToolset.Extensibility/BaseWindowsInstallerBackendBinderExtension.cs
index e2be63bf..c0086aed 100644
--- a/src/WixToolset.Extensibility/BaseWindowsInstallerBackendBinderExtension.cs
+++ b/src/WixToolset.Extensibility/BaseWindowsInstallerBackendBinderExtension.cs
@@ -53,6 +53,13 @@ namespace WixToolset.Extensibility
         }
 
         /// <summary>
+        /// See <see cref="IWindowsInstallerBackendBinderExtension.FullyResolved(IntermediateSection)"/>
+        /// </summary>
+        public virtual void FullyResolved(IntermediateSection section)
+        {
+        }
+
+        /// <summary>
         /// See <see cref="IWindowsInstallerBackendBinderExtension.PreBackendBind(IBindContext)"/>
         /// </summary>
         public virtual IResolvedCabinet ResolveCabinet(string cabinetPath, IEnumerable<IBindFileWithPath> files) => null;
diff --git a/src/WixToolset.Extensibility/IWindowsInstallerBackendBinderExtension.cs b/src/WixToolset.Extensibility/IWindowsInstallerBackendBinderExtension.cs
index f257f1ec..b913dadc 100644
--- a/src/WixToolset.Extensibility/IWindowsInstallerBackendBinderExtension.cs
+++ b/src/WixToolset.Extensibility/IWindowsInstallerBackendBinderExtension.cs
@@ -13,26 +13,53 @@ namespace WixToolset.Extensibility
     /// </summary>
     public interface IWindowsInstallerBackendBinderExtension
     {
-#pragma warning disable 1591 // TODO: add documentation
+        /// <summary>
+        /// Table definitions provided by the extension.
+        /// </summary>
         IEnumerable<TableDefinition> TableDefinitions { get; }
-#pragma warning restore 1591
 
         /// <summary>
         /// Called before binding occurs.
         /// </summary>
         void PreBackendBind(IBindContext context);
 
-#pragma warning disable 1591 // TODO: add documentation
+        /// <summary>
+        /// 
+        /// </summary>
+        /// <param name="section">The resolved intermedate section.</param>
+        void FullyResolved(IntermediateSection section);
+
+        /// <summary>
+        /// Finds an existing cabinet that contains the provided files.
+        /// </summary>
+        /// <param name="cabinetPath">Path to the cabinet.</param>
+        /// <param name="files">Files contained in the cabinet.</param>
+        /// <returns>Resolved cabinet options or null if the cabinet could not be found.</returns>
         IResolvedCabinet ResolveCabinet(string cabinetPath, IEnumerable<IBindFileWithPath> files);
 
-        string ResolveMedia(MediaSymbol mediaRow, string mediaLayoutDirectory, string layoutDirectory);
+        /// <summary>
+        /// Override layout location for a media.
+        /// </summary>
+        /// <param name="mediaSymbol">Media symbol.</param>
+        /// <param name="mediaLayoutDirectory">Default media layout directory.</param>
+        /// <param name="layoutDirectory">Default layout directory.</param>
+        /// <returns>Layout location or null to use the default processing.</returns>
+        string ResolveMedia(MediaSymbol mediaSymbol, string mediaLayoutDirectory, string layoutDirectory);
 
+        /// <summary>
+        /// 
+        /// </summary>
+        /// <param name="section"></param>
+        /// <param name="symbol"></param>
+        /// <param name="output">Windows Installer data </param>
+        /// <param name="tableDefinitions">Collection of table definitions available for the output.</param>
+        /// <returns>True if the symbol was added to the output, or false if not.</returns>
         bool TryAddSymbolToOutput(IntermediateSection section, IntermediateSymbol symbol, WindowsInstallerData output, TableDefinitionCollection tableDefinitions);
-#pragma warning restore 1591
 
         /// <summary>
         /// Called after all output changes occur and right before the output is bound into its final format.
         /// </summary>
+        /// <param name="result">Bind result to process.</param>
         void PostBackendBind(IBindResult result);
     }
 }