Update Reference Sources to .NET Framework 4.6

11 files changed, 37 insertions, 37 deletions

diff --git a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/MenuAdapter.cs b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/MenuAdapter.cs
index 83c46ef3715..2fc4fce2058 100644
--- a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/MenuAdapter.cs
+++ b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/MenuAdapter.cs
@@ -44,8 +44,8 @@ namespace System.Web.UI.WebControls.Adapters {
 
         private string Escape(string path) {
             // This function escapes \\ so that they don't get replaced because of
-            // a Netscape 4 bug. Other escapable characters will be escaped by .
-            // _ becomes __ and \\ becomes \_\
+            // a Netscape 4 
+
             StringBuilder b = null;
 
             if (String.IsNullOrEmpty(path)) {
diff --git a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlBulletedListAdapter.cs b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlBulletedListAdapter.cs
index c30735b2563..711bedb6551 100644
--- a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlBulletedListAdapter.cs
+++ b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlBulletedListAdapter.cs
@@ -32,7 +32,7 @@ namespace System.Web.UI.WebControls.Adapters {
                     writer.WriteBreak();
                     break;
                 case BulletedListDisplayMode.HyperLink:
-                    // 
+                    // TODO: if index == 0, set accesskey.  Needs a new RenderBeginHyperlink method.
                     string targetURL = Control.ResolveClientUrl(items[index].Value);
                     if (items[index].Enabled) {
                         PageAdapter.RenderBeginHyperlink(writer, targetURL, true /* encode */, items[index].Text);
@@ -45,7 +45,7 @@ namespace System.Web.UI.WebControls.Adapters {
                     break;
                 case BulletedListDisplayMode.LinkButton:
                     if (items[index].Enabled) {
-                        // 
+                        // TODO: if index == 0, set accesskey.  Needs a new RenderPostBackEvent method.               
                         PageAdapter.RenderPostBackEvent(writer, Control.UniqueID, index.ToString(CultureInfo.InvariantCulture),
                                                             items[index].Text, items[index].Text);
                     } else {
diff --git a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlDataBoundLiteralControlAdapter.cs b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlDataBoundLiteralControlAdapter.cs
index f1f5c99a3ca..4a4fd4232b7 100644
--- a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlDataBoundLiteralControlAdapter.cs
+++ b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlDataBoundLiteralControlAdapter.cs
@@ -21,8 +21,8 @@ namespace System.Web.UI.Adapters {
         protected internal override void EndRender(HtmlTextWriter writer) {
         }
         
-        // 
-
+        // BUGBUG: This override is for compatibility with MMIT.
+        // MMIT legacy pages also use this adapter -UNDONE: Review once MMIT legacy plan is complete.
         protected internal override void Render(HtmlTextWriter writer) {
             WmlTextWriter wmlWriter = writer as WmlTextWriter;
             if (wmlWriter == null) {
diff --git a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlHyperLinkAdapter.cs b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlHyperLinkAdapter.cs
index 9c919c17b52..b80aa4f9de2 100644
--- a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlHyperLinkAdapter.cs
+++ b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlHyperLinkAdapter.cs
@@ -20,8 +20,8 @@ namespace System.Web.UI.WebControls.Adapters {
 
             String text = Control.Text;
             if (text.Length == 0) {
-                // Whidbey 18195 
-
+                // Whidbey 18195 UNDONE: This solution is somewhat ad hoc, awaiting spec resolution on IStaticTextControl
+                // in M2.  For now, take text from first IStaticTextControl or DataboundLiteralControl.
                 foreach(Control child in Control.Controls) {
                     if (child is IStaticTextControl) {
                         text = ((IStaticTextControl)child).Text;
diff --git a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlImageButtonAdapter.cs b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlImageButtonAdapter.cs
index 31d463449da..f98b86743e9 100644
--- a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlImageButtonAdapter.cs
+++ b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlImageButtonAdapter.cs
@@ -10,9 +10,9 @@ namespace System.Web.UI.WebControls.Adapters {
     using System.Web.UI.Adapters;
     using System.Web.UI.WebControls;
 
-    // 
-
-
+    // REVIEW: Inheritance.  If this inherits from ImageButtonAdapter, there is no way to create a
+    // WmlImageAdapter and set the Control property to delegate rendering (base.Render, below). Control is read-only. 
+    // Maybe Control should be get/set for this situation.
     public class WmlImageButtonAdapter : WmlImageAdapter {
 
         protected new ImageButton Control {
@@ -30,7 +30,7 @@ namespace System.Web.UI.WebControls.Adapters {
                 postUrl = Control.ResolveClientUrl (Control.PostBackUrl);
             }
 
-            // 
+            // UNDONE: Replace hard coded string indexer with strongly typed capability.
             if (Page != null && Page.Request != null && (String)Page.Request.Browser["supportsImageSubmit"] == "false") {
                 writer.EnterStyle(Control.ControlStyle);
 
diff --git a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlLiteralAdapter.cs b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlLiteralAdapter.cs
index f5f05650e59..01845963870 100644
--- a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlLiteralAdapter.cs
+++ b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlLiteralAdapter.cs
@@ -21,7 +21,7 @@ namespace System.Web.UI.WebControls.Adapters {
         protected internal override void EndRender(HtmlTextWriter writer) {
         }
 
-        // 
+        // BUGBUG: This override is for compatibility with MMIT only. -UNDONE: Review once MMIT legacy plan is complete.
         protected internal override void Render(HtmlTextWriter writer) {
             WmlTextWriter wmlWriter = writer as WmlTextWriter;
             if (wmlWriter == null) {
diff --git a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlLiteralControlAdapter.cs b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlLiteralControlAdapter.cs
index 7cf14fc7538..052a41fe32a 100644
--- a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlLiteralControlAdapter.cs
+++ b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlLiteralControlAdapter.cs
@@ -20,8 +20,8 @@ namespace System.Web.UI.Adapters {
         protected internal override void EndRender(HtmlTextWriter writer) {
         }
 
-        // 
-
+        // BUGBUG: This override is for compatibility with MMIT.
+        // MMIT legacy pages also use this adapter -UNDONE: Review once MMIT legacy plan is complete.
         protected internal override void Render(HtmlTextWriter writer) {
             WmlTextWriter wmlWriter = writer as WmlTextWriter;
             if (wmlWriter == null) {
diff --git a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlPageAdapter.cs b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlPageAdapter.cs
index 87fcb974b51..08636d4e370 100644
--- a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlPageAdapter.cs
+++ b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlPageAdapter.cs
@@ -67,8 +67,8 @@ namespace System.Web.UI.Adapters {
             }
         }
 
-        // 
-
+        // UNDONE: Internal because needed by WmlTextWriter.  Consider removing this prop somehow.
+        // Returns true if form variables have been written.
         internal bool WrittenFormVariables {
             get {
                 return _writtenFormVariables;
@@ -131,8 +131,8 @@ namespace System.Web.UI.Adapters {
             return collection;
         }
 
-        // 
-
+        // UNDONE: For M1, we only have Wml browsers which do not support accesskey.  For later milestones, make this
+        // dependent on a capability or replace with a capability.
         private bool DoesBrowserSupportAccessKey() {
             return false;
         }
@@ -235,16 +235,16 @@ namespace System.Web.UI.Adapters {
         }
 
         // Renders the beginning of the form.
-        // 
+        // UNDONE: Remove internal modifier when method is completely removed from writer.
         protected internal virtual void RenderBeginForm(WmlTextWriter writer) {
 
             RenderBeginCardTag(writer);
 
             // Write form variables.
 
-            // 
-
-
+            // UNDONE: Move writer._provideBackButton to this adapter.
+            // Review: In V1 we had a writer.ProvideBackButton property, is there any need for this with (more advanced)
+            // whidbey devices?
             _writtenFormVariables = true;
             if (_formVariables == null) {
                 _formVariables = new ListDictionary();
@@ -259,7 +259,7 @@ namespace System.Web.UI.Adapters {
             RenderSetFormVariables(writer);
             RenderPostUrlFormVariable(writer);
             writer.WriteLine("</refresh></onevent>");
-            // 
+            // UNDONE: formAdapter.RenderExtraCardElements(this);
             writer.BeginFormOrPanel();
         }
 
@@ -432,7 +432,7 @@ namespace System.Web.UI.Adapters {
             RenderTargetAndArgumentPostFields(writer, target, argument, postFieldType);
             RenderPostFieldVariableDictionary(writer, _dynamicPostFields);
             RenderPostFieldDictionary(writer, _staticPostFields);
-            // 
+            // UNDONE: Add postbacks for variables which are not on the current page.
             writer.WriteEndTag("go");
         }
 
@@ -442,8 +442,8 @@ namespace System.Web.UI.Adapters {
                 return;
             }
             writer.Write("?");
-            // 
-
+            // UNDONE: MMIT IPageAdapter.PersistCookielessData NYI
+            // if(Page.Adapter.PersistCookielessData && Browser["canRenderOneventAndPrevElementsTogether"] != "false")
             if (!StringUtil.EqualsIgnoreCase((string)Browser["canRenderOneventAndPrevElementsTogether"], "false")) {
                 queryString = writer.ReplaceFormsCookieWithVariable(queryString);
             }
@@ -504,7 +504,7 @@ namespace System.Web.UI.Adapters {
 
             writer.Write("<onevent type=\"onenterforward\">");
             RenderFormPostInGoAction(writer, null, _postBackEventArgumentVarName, WmlPostFieldType.Variable, String.Empty);
-            // 
+            // REVIEW: Should we always include page hidden variables.
             writer.WriteLine("</onevent>");
 
             writer.WriteLine("<onevent type=\"onenterbackward\"><prev /></onevent>");
diff --git a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlPhoneLinkAdapter.cs b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlPhoneLinkAdapter.cs
index c4dfaa39a26..efccabe639b 100644
--- a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlPhoneLinkAdapter.cs
+++ b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlPhoneLinkAdapter.cs
@@ -11,7 +11,7 @@ namespace System.Web.UI.WebControls.Adapters {
 
     public class WmlPhoneLinkAdapter : PhoneLinkAdapter {
 
-        // 
+        // UNDONE: Add style.
         protected internal override void Render(HtmlTextWriter markupWriter) {
             WmlTextWriter writer = (WmlTextWriter)markupWriter;
             String text, url, phoneNumber;
@@ -42,7 +42,7 @@ namespace System.Web.UI.WebControls.Adapters {
             if (Page != null && Page.Request != null) {
                 browser = Page.Request.Browser;
             }
-            // 
+            // TODO: Replace hard coded string key.
             if (browser != null && (String)browser["canInitiateVoiceCall"] != "true") {
                 text = String.Format(controlText,
                                      originalNumber);
@@ -54,7 +54,7 @@ namespace System.Web.UI.WebControls.Adapters {
                 // showing as text so it can be selected.  If it is not
                 // formatted in the text yet, append it to the end of the
                 // text.
-                // 
+                // TODO: Replace hard coded string key.
                 if (browser != null && browser["requiresPhoneNumbersAsPlainText"] == "true") {
                     text = controlText + " " + phoneNumber;
                     url = String.Empty;
diff --git a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlRadioButtonAdapter.cs b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlRadioButtonAdapter.cs
index 397b021374f..7923294a176 100644
--- a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlRadioButtonAdapter.cs
+++ b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlRadioButtonAdapter.cs
@@ -141,14 +141,14 @@ namespace System.Web.UI.WebControls.Adapters {
         // RenderAsGroup returns a RadioButtonGroup object if the group should be 
         // rendered in a single <select> statement, or null if autopostback should
         // be enabled.
-        // 
+        // UNDONE: Check more general case when radiobuttons do not all share the same parent
         private RadioButtonGroup RenderAsGroup(RadioButton r) {
             bool startedSequence = false;
             bool finishedSequence = false;
 
             RadioButtonGroup group = new RadioButtonGroup();
 
-            //
+            //TODO : Check all controls on page
             foreach (Control c in r.Parent.Controls) {
                 RadioButton radioSibling = c as RadioButton;
                 LiteralControl literalSibling = c as LiteralControl;
diff --git a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlTextBoxAdapter.cs b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlTextBoxAdapter.cs
index a3875b40a56..51124c6afd0 100644
--- a/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlTextBoxAdapter.cs
+++ b/mcs/class/referencesource/System.Web/UI/WebControls/Adapters/WmlTextBoxAdapter.cs
@@ -33,7 +33,7 @@ namespace System.Web.UI.WebControls.Adapters {
 
             if (Control.TextMode == TextBoxMode.Password) {
                 value = String.Empty;
-                // 
+                // UNDONE: Consider adding property or wml adapter property to set requiresRandomID to true.
                 requiresRandomID = true;
             }
 
@@ -53,8 +53,8 @@ namespace System.Web.UI.WebControls.Adapters {
                 writer.MapClientIDToShortName(Control.ClientID, requiresRandomID);
             }
 
-            // 
-
+            // UNDONE: There are some wml-specific properties (format, title, size, maxLength) which
+            // we should consider for Whidbey.
             RenderTextBox((WmlTextWriter)writer, Control.ClientID, 
                           value,
                           null /* format */, 
@@ -69,7 +69,7 @@ namespace System.Web.UI.WebControls.Adapters {
         // Renders the TextBox.
         public virtual void RenderTextBox(WmlTextWriter writer, String id, String value, String format, String title, bool password, int size, int maxLength, bool generateRandomID) {
             if (!writer.AnalyzeMode) {
-                // 
+                // UNDONE: Handle rendersBreakBeforeWmlSelectAndInput, if this capability is still needed for Whidbey devices                
                                 
                 // VSWhidbey 147458.  Close any style tags.
                 writer.CloseCurrentStyleTags();