]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Universal/SetupBrowserDxe/Ui.c
Based on the feature PCD value, browser will decide whether to gray out the read...
[mirror_edk2.git] / MdeModulePkg / Universal / SetupBrowserDxe / Ui.c
index f3019a84084b2a5d261ff522d75c7cb011061198..e9f9f70edac4063a9871a7a9b4c22af6943cd851 100644 (file)
@@ -1,7 +1,7 @@
 /** @file\r
 Utility functions for User Interface functions.\r
 \r
-Copyright (c) 2004 - 2012, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2004 - 2013, Intel Corporation. All rights reserved.<BR>\r
 This program and the accompanying materials\r
 are licensed and made available under the terms and conditions of the BSD License\r
 which accompanies this distribution.  The full text of the license may be found at\r
@@ -778,6 +778,9 @@ UiAddMenuOption (
     if ((Statement->ValueExpression != NULL) ||\r
         ((Statement->QuestionFlags & EFI_IFR_FLAG_READ_ONLY) != 0)) {\r
       MenuOption->ReadOnly = TRUE;\r
+      if (FeaturePcdGet (PcdBrowerGrayOutReadOnlyMenu)) {\r
+        MenuOption->GrayOut = TRUE;\r
+      }\r
     }\r
 \r
     InsertTailList (&gMenuOption, &MenuOption->Link);\r
@@ -1455,7 +1458,7 @@ GetLineByWidth (
     // Skip the space info at the begin of next line.\r
     //  \r
     *Index = (UINT16) (*Index + StrOffset + 1);\r
-  } else if ((InputString[*Index + StrOffset] == CHAR_LINEFEED)) {\r
+  } else if (InputString[*Index + StrOffset] == CHAR_LINEFEED) {\r
     //\r
     // Skip the /n or /n/r info.\r
     //\r
@@ -1464,7 +1467,7 @@ GetLineByWidth (
     } else {\r
       *Index = (UINT16) (*Index + StrOffset + 1);\r
     }\r
-  } else if ((InputString[*Index + StrOffset] == CHAR_CARRIAGE_RETURN)) {\r
+  } else if (InputString[*Index + StrOffset] == CHAR_CARRIAGE_RETURN) {\r
     //\r
     // Skip the /r or /r/n info.\r
     //  \r
@@ -1507,36 +1510,34 @@ UpdateOptionSkipLines (
 \r
   Row           = 0;\r
   OptionString  = NULL;\r
+  Width         = (UINT16) gOptionBlockWidth;\r
+  OriginalRow   = 0;\r
+  GlyphWidth    = 1;\r
+  \r
   ProcessOptions (Selection, MenuOption, FALSE, &OptionString);\r
+  if (OptionString == NULL) {\r
+    return;\r
+  }\r
 \r
-  if (OptionString != NULL) {\r
-    Width               = (UINT16) gOptionBlockWidth;\r
-\r
-    OriginalRow         = Row;\r
-    GlyphWidth          = 1;\r
-\r
-    for (Index = 0; GetLineByWidth (OptionString, Width, &GlyphWidth, &Index, &OutputString) != 0x0000;) {\r
+  for (Index = 0; GetLineByWidth (OptionString, Width, &GlyphWidth, &Index, &OutputString) != 0x0000;) {\r
+    //\r
+    // If there is more string to process print on the next row and increment the Skip value\r
+    //\r
+    if (StrLen (&OptionString[Index]) != 0) {\r
+      Row++;\r
       //\r
-      // If there is more string to process print on the next row and increment the Skip value\r
+      // Since the Number of lines for this menu entry may or may not be reflected accurately\r
+      // since the prompt might be 1 lines and option might be many, and vice versa, we need to do\r
+      // some testing to ensure we are keeping this in-sync.\r
       //\r
-      if (StrLen (&OptionString[Index]) != 0) {\r
-        Row++;\r
-        //\r
-        // Since the Number of lines for this menu entry may or may not be reflected accurately\r
-        // since the prompt might be 1 lines and option might be many, and vice versa, we need to do\r
-        // some testing to ensure we are keeping this in-sync.\r
-        //\r
-        // If the difference in rows is greater than or equal to the skip value, increase the skip value\r
-        //\r
-        if ((Row - OriginalRow) >= MenuOption->Skip) {\r
-          MenuOption->Skip++;\r
-        }\r
+      // If the difference in rows is greater than or equal to the skip value, increase the skip value\r
+      //\r
+      if ((Row - OriginalRow) >= MenuOption->Skip) {\r
+        MenuOption->Skip++;\r
       }\r
-\r
-      FreePool (OutputString);\r
     }\r
 \r
-    Row = OriginalRow;\r
+    FreePool (OutputString);\r
   }\r
 \r
   if (OptionString != NULL) {\r
@@ -1612,6 +1613,10 @@ ValueIsScroll (
 \r
   @return The row distance from current MenuOption to next selectable MenuOption.\r
 \r
+  @retval -1       Reach the begin of the menu, still can't find the selectable menu.\r
+  @retval Value    Find the selectable menu, maybe the truly selectable, maybe the l\r
+                   last menu showing at current form.\r
+\r
 **/\r
 INTN\r
 MoveToNextStatement (\r
@@ -1632,41 +1637,56 @@ MoveToNextStatement (
 \r
   while (TRUE) {\r
     NextMenuOption = MENU_OPTION_FROM_LINK (Pos);\r
+    //\r
+    // NextMenuOption->Row == 0 means this menu has not calculate\r
+    // the NextMenuOption->Skip value yet, just calculate here.\r
+    //\r
     if (NextMenuOption->Row == 0) {\r
       UpdateOptionSkipLines (Selection, NextMenuOption);\r
     }\r
     \r
     if (GoUp && (PreMenuOption != NextMenuOption)) {\r
       //\r
-      // Current Position doesn't need to be caculated when go up.\r
-      // Caculate distanct at first when go up\r
+      // In this case, still can't find the selectable menu,\r
+      // return the last one in the showing form.\r
       //\r
       if ((UINTN) Distance + NextMenuOption->Skip > GapToTop) {\r
         NextMenuOption = PreMenuOption;\r
         break;\r
       }\r
+\r
+      //\r
+      // Current Position doesn't need to be caculated when go up.\r
+      // Caculate distanct at first when go up\r
+      //      \r
       Distance += NextMenuOption->Skip;\r
     }\r
+\r
     if (IsSelectable (NextMenuOption)) {\r
       break;\r
     }\r
+\r
+    //\r
+    // Arrive at begin of the menu list.\r
+    //\r
     if ((GoUp ? Pos->BackLink : Pos->ForwardLink) == &gMenuOption) {\r
-      //\r
-      // Arrive at top.\r
-      //\r
       Distance = -1;\r
       break;\r
     }\r
+\r
     if (!GoUp) {\r
       //\r
-      // Caculate distanct at later when go down\r
+      // In this case, still can't find the selectable menu,\r
+      // return the last one in the showing form.\r
       //\r
       if ((UINTN) Distance + NextMenuOption->Skip > GapToTop) {\r
         NextMenuOption = PreMenuOption;\r
         break;\r
       }\r
+\r
       Distance += NextMenuOption->Skip;\r
     }\r
+\r
     PreMenuOption = NextMenuOption;\r
     Pos = (GoUp ? Pos->BackLink : Pos->ForwardLink);\r
   }\r
@@ -1953,55 +1973,6 @@ FormSetGuidToHiiHandle (
   return HiiHandle;\r
 }\r
 \r
-/**\r
-  Transfer the device path string to binary format.\r
-\r
-  @param   StringPtr     The device path string info.\r
-\r
-  @retval  Device path binary info.\r
-\r
-**/\r
-EFI_DEVICE_PATH_PROTOCOL *\r
-ConvertDevicePathFromText (\r
-  IN CHAR16  *StringPtr\r
-  )\r
-{\r
-  UINTN                           BufferSize;\r
-  EFI_DEVICE_PATH_PROTOCOL        *DevicePath;\r
-  CHAR16                          TemStr[2];\r
-  UINT8                           *DevicePathBuffer;\r
-  UINTN                           Index;\r
-  UINT8                           DigitUint8;\r
-\r
-  ASSERT (StringPtr != NULL);\r
-\r
-  BufferSize = StrLen (StringPtr) / 2;\r
-  DevicePath = AllocatePool (BufferSize);\r
-  ASSERT (DevicePath != NULL);\r
-  \r
-  //\r
-  // Convert from Device Path String to DevicePath Buffer in the reverse order.\r
-  //\r
-  DevicePathBuffer = (UINT8 *) DevicePath;\r
-  for (Index = 0; StringPtr[Index] != L'\0'; Index ++) {\r
-    TemStr[0] = StringPtr[Index];\r
-    DigitUint8 = (UINT8) StrHexToUint64 (TemStr);\r
-    if (DigitUint8 == 0 && TemStr[0] != L'0') {\r
-      //\r
-      // Invalid Hex Char as the tail.\r
-      //\r
-      break;\r
-    }\r
-    if ((Index & 1) == 0) {\r
-      DevicePathBuffer [Index/2] = DigitUint8;\r
-    } else {\r
-      DevicePathBuffer [Index/2] = (UINT8) ((DevicePathBuffer [Index/2] << 4) + DigitUint8);\r
-    }\r
-  }\r
-\r
-  return DevicePath;\r
-}\r
-\r
 /**\r
   Process the goto op code, update the info in the selection structure.\r
 \r
@@ -2047,16 +2018,32 @@ ProcessGotoOpCode (
     if (Selection->Form->ModalForm) {\r
       return Status;\r
     }\r
+\r
     //\r
     // Goto another Hii Package list\r
     //\r
-    Selection->Action = UI_ACTION_REFRESH_FORMSET;\r
-    DevicePath = ConvertDevicePathFromText (StringPtr);\r
-\r
-    Selection->Handle = DevicePathToHiiHandle (DevicePath);\r
-    FreePool (DevicePath);\r
-    FreePool (StringPtr);\r
+    if (mPathFromText != NULL) {\r
+      DevicePath = mPathFromText->ConvertTextToDevicePath(StringPtr);\r
+      if (DevicePath != NULL) {\r
+        Selection->Handle = DevicePathToHiiHandle (DevicePath);\r
+        FreePool (DevicePath);\r
+      }\r
+      FreePool (StringPtr);\r
+    } else {\r
+      //\r
+      // Not found the EFI_DEVICE_PATH_FROM_TEXT_PROTOCOL protocol.\r
+      //\r
+      do {\r
+        CreateDialog (4, TRUE, 0, NULL, &Key, gEmptyString, gProtocolNotFound, gPressEnter, gEmptyString);\r
+      } while (Key.UnicodeChar != CHAR_CARRIAGE_RETURN);\r
+      if (Repaint != NULL) {\r
+        *Repaint = TRUE;\r
+      }\r
+      FreePool (StringPtr);\r
+      return Status;\r
+    }\r
 \r
+    Selection->Action = UI_ACTION_REFRESH_FORMSET;\r
     if (Selection->Handle == NULL) {\r
       //\r
       // If target Hii Handle not found, exit\r
@@ -2173,7 +2160,6 @@ UiDisplayMenu (
 {\r
   INTN                            SkipValue;\r
   INTN                            Difference;\r
-  INTN                            OldSkipValue;\r
   UINTN                           DistanceValue;\r
   UINTN                           Row;\r
   UINTN                           Col;\r
@@ -2254,7 +2240,6 @@ UiDisplayMenu (
   UpArrow             = FALSE;\r
   DownArrow           = FALSE;\r
   SkipValue           = 0;\r
-  OldSkipValue        = 0;\r
   MenuRefreshEntry    = gMenuRefreshHead;\r
 \r
   NextMenuOption      = NULL;\r
@@ -2355,6 +2340,9 @@ UiDisplayMenu (
         Temp            = (UINTN) SkipValue;\r
         Temp2           = (UINTN) SkipValue;\r
 \r
+        //\r
+        // 1. Clear the screen.\r
+        //\r
         if (Selection->Form->ModalForm) {\r
           ClearLines (\r
             LocalScreen.LeftColumn + ModalSkipColumn,\r
@@ -2375,6 +2363,9 @@ UiDisplayMenu (
         UiFreeRefreshList ();\r
         MinRefreshInterval = 0;\r
 \r
+        //\r
+        // 2.Paint the menu.\r
+        //\r
         for (Link = TopOfScreen; Link != &gMenuOption; Link = Link->ForwardLink) {\r
           MenuOption          = MENU_OPTION_FROM_LINK (Link);\r
           MenuOption->Row     = Row;\r
@@ -2414,7 +2405,13 @@ UiDisplayMenu (
               );\r
           }\r
 \r
+          //\r
+          // 2.1. Paint the description.\r
+          //\r
           for (Index = 0; GetLineByWidth (MenuOption->Description, Width, &GlyphWidth, &Index, &OutputString) != 0x0000;) {\r
+            //\r
+            // Temp means need to skip how many lines from the start.\r
+            //\r
             if ((Temp == 0) && (Row <= BottomRow)) {\r
               PrintStringAt (MenuOption->Col, Row, OutputString);\r
             }\r
@@ -2436,6 +2433,9 @@ UiDisplayMenu (
           Temp  = 0;\r
           Row   = OriginalRow;\r
 \r
+          //\r
+          // 2.2. Paint the option string.\r
+          //\r
           Status = ProcessOptions (Selection, MenuOption, FALSE, &OptionString);\r
           if (EFI_ERROR (Status)) {\r
             //\r
@@ -2492,7 +2492,7 @@ UiDisplayMenu (
           }\r
 \r
           //\r
-          // If Question has refresh guid, register the op-code.\r
+          // 2.4 Special process for Test opcode with test two.\r
           //\r
           if (!CompareGuid (&Statement->RefreshGuid, &gZeroGuid)) {\r
             if (gMenuEventGuidRefreshHead == NULL) {\r
@@ -2600,11 +2600,10 @@ UiDisplayMenu (
           gST->ConOut->SetAttribute (gST->ConOut, PcdGet8 (PcdBrowserFieldTextColor) | FIELD_BACKGROUND);\r
 \r
           //\r
-          // Need to handle the bottom of the display\r
+          // 3. Update the row info which will be used by next menu.\r
           //\r
-          if (MenuOption->Skip > 1) {\r
+          if (Link == TopOfScreen) {\r
             Row += MenuOption->Skip - SkipValue;\r
-            SkipValue = 0;\r
           } else {\r
             Row += MenuOption->Skip;\r
           }\r
@@ -2656,6 +2655,16 @@ UiDisplayMenu (
       // NewPos:     Current menu option that need to hilight\r
       //\r
       ControlFlag = CfUpdateHelpString;\r
+      if (TopOfScreen == &MenuOption->Link) {\r
+        Temp = SkipValue;\r
+      } else {\r
+        Temp = 0;\r
+      }\r
+      if (NewPos == TopOfScreen) {\r
+        Temp2 = SkipValue;\r
+      } else {\r
+        Temp2 = 0;\r
+      }\r
       if (InitializedFlag) {\r
         InitializedFlag = FALSE;\r
         MoveToNextStatement (Selection, FALSE, &NewPos, BottomRow - TopRow);\r
@@ -2688,7 +2697,7 @@ UiDisplayMenu (
             SavedMenuOption = MENU_OPTION_FROM_LINK (Link);\r
             Index += SavedMenuOption->Skip;\r
             if (Link == TopOfScreen) {\r
-              Index -= OldSkipValue;\r
+              Index -= SkipValue;\r
             }\r
             Link = Link->ForwardLink;\r
           }\r
@@ -2746,7 +2755,6 @@ UiDisplayMenu (
               //\r
               SkipValue    = 0;\r
               TopOfScreen  = Link;\r
-              OldSkipValue = SkipValue;\r
             } else {\r
               //\r
               // Check whether need to skip some line for menu shows at the top of the page.\r
@@ -2754,7 +2762,6 @@ UiDisplayMenu (
               SkipValue = Index - BottomRow - 1;\r
               if (SkipValue > 0 && SkipValue < (INTN) SavedMenuOption->Skip) {\r
                 TopOfScreen     = Link;\r
-                OldSkipValue    = SkipValue;\r
               } else {\r
                 SkipValue       = 0;\r
                 TopOfScreen     = Link->ForwardLink;\r
@@ -2796,17 +2803,22 @@ UiDisplayMenu (
             GlyphWidth          = 1;\r
 \r
             for (Index = 0; GetLineByWidth (OptionString, Width, &GlyphWidth, &Index, &OutputString) != 0x0000;) {\r
-              if (MenuOption->Row >= TopRow && MenuOption->Row <= BottomRow) {\r
+              if ((Temp == 0) && (MenuOption->Row >= TopRow) && (MenuOption->Row <= BottomRow)) {\r
                 PrintStringAt (MenuOption->OptCol, MenuOption->Row, OutputString);\r
               }\r
               //\r
               // If there is more string to process print on the next row and increment the Skip value\r
               //\r
               if (StrLen (&OptionString[Index]) != 0) {\r
-                MenuOption->Row++;\r
+                if (Temp == 0) {\r
+                  MenuOption->Row++;\r
+                }\r
               }\r
 \r
               FreePool (OutputString);\r
+              if (Temp != 0) {\r
+                Temp--;\r
+              }\r
             }\r
 \r
             MenuOption->Row = OriginalRow;\r
@@ -2825,17 +2837,22 @@ UiDisplayMenu (
               GlyphWidth  = 1;\r
 \r
               for (Index = 0; GetLineByWidth (MenuOption->Description, Width, &GlyphWidth, &Index, &OutputString) != 0x0000;) {\r
-                if (MenuOption->Row >= TopRow && MenuOption->Row <= BottomRow) {\r
+                if ((Temp == 0) && (MenuOption->Row >= TopRow) && (MenuOption->Row <= BottomRow)) {\r
                   PrintStringAt (MenuOption->Col, MenuOption->Row, OutputString);\r
                 }\r
                 //\r
                 // If there is more string to process print on the next row and increment the Skip value\r
                 //\r
                 if (StrLen (&MenuOption->Description[Index]) != 0) {\r
-                  MenuOption->Row++;\r
+                  if (Temp == 0) {\r
+                    MenuOption->Row++;\r
+                  }\r
                 }\r
 \r
                 FreePool (OutputString);\r
+                if (Temp != 0) {\r
+                  Temp--;\r
+                }\r
               }\r
 \r
               MenuOption->Row = OriginalRow;\r
@@ -2897,17 +2914,22 @@ UiDisplayMenu (
           GlyphWidth          = 1;\r
 \r
           for (Index = 0; GetLineByWidth (OptionString, Width, &GlyphWidth, &Index, &OutputString) != 0x0000;) {\r
-            if (MenuOption->Row >= TopRow && MenuOption->Row <= BottomRow) {\r
+            if ((Temp2 == 0) && (MenuOption->Row >= TopRow) && (MenuOption->Row <= BottomRow) ) {\r
               PrintStringAt (MenuOption->OptCol, MenuOption->Row, OutputString);\r
             }\r
             //\r
             // If there is more string to process print on the next row and increment the Skip value\r
             //\r
             if (StrLen (&OptionString[Index]) != 0) {\r
+              if (Temp2 == 0) {\r
               MenuOption->Row++;\r
+              }\r
             }\r
 \r
             FreePool (OutputString);\r
+            if (Temp2 != 0) {\r
+              Temp2--;\r
+            }\r
           }\r
 \r
           MenuOption->Row = OriginalRow;\r
@@ -2921,17 +2943,22 @@ UiDisplayMenu (
             GlyphWidth          = 1;\r
 \r
             for (Index = 0; GetLineByWidth (MenuOption->Description, Width, &GlyphWidth, &Index, &OutputString) != 0x0000;) {\r
-              if (MenuOption->Row >= TopRow && MenuOption->Row <= BottomRow) {\r
+              if ((Temp2 == 0) && (MenuOption->Row >= TopRow) && (MenuOption->Row <= BottomRow) ) {\r
                 PrintStringAt (MenuOption->Col, MenuOption->Row, OutputString);\r
               }\r
               //\r
               // If there is more string to process print on the next row and increment the Skip value\r
               //\r
               if (StrLen (&MenuOption->Description[Index]) != 0) {\r
-                MenuOption->Row++;\r
+                if (Temp2 == 0) {\r
+                  MenuOption->Row++;\r
+                }\r
               }\r
 \r
               FreePool (OutputString);\r
+              if (Temp2 != 0) {\r
+                Temp2--;\r
+              }\r
             }\r
 \r
             MenuOption->Row = OriginalRow;\r
@@ -3110,25 +3137,33 @@ UiDisplayMenu (
       //\r
       // Wait for user's selection\r
       //\r
-      do {\r
-        Status = UiWaitForSingleEvent (gST->ConIn->WaitForKey, 0, MinRefreshInterval);\r
-      } while (Status == EFI_TIMEOUT);\r
+      while (TRUE) {\r
+        Status = gST->ConIn->ReadKeyStroke (gST->ConIn, &Key);\r
+        if (!EFI_ERROR (Status)) {\r
+          break;\r
+        }\r
 \r
-      if (Selection->Action == UI_ACTION_REFRESH_FORMSET) {\r
         //\r
-        // IFR is updated in Callback of refresh opcode, re-parse it\r
+        // If we encounter error, continue to read another key in.\r
         //\r
-        ControlFlag = CfCheckSelection;\r
-        Selection->Statement = NULL;\r
-        break;\r
+        if (Status != EFI_NOT_READY) {\r
+          continue;\r
+        }\r
+\r
+        Status = UiWaitForSingleEvent (gST->ConIn->WaitForKey, 0, MinRefreshInterval);\r
+        ASSERT_EFI_ERROR (Status);\r
+\r
+        if (Selection->Action == UI_ACTION_REFRESH_FORMSET) {\r
+          //\r
+          // IFR is updated in Callback of refresh opcode, re-parse it\r
+          //\r
+          ControlFlag = CfCheckSelection;\r
+          Selection->Statement = NULL;\r
+          break;\r
+        }\r
       }\r
 \r
-      Status = gST->ConIn->ReadKeyStroke (gST->ConIn, &Key);\r
-      //\r
-      // If we encounter error, continue to read another key in.\r
-      //\r
-      if (EFI_ERROR (Status)) {\r
-        ControlFlag = CfReadKey;\r
+      if (ControlFlag == CfCheckSelection) {\r
         break;\r
       }\r
 \r
@@ -3438,7 +3473,6 @@ UiDisplayMenu (
           TopOfScreen = NewPos;\r
           Repaint     = TRUE;\r
           SkipValue = 0;\r
-          OldSkipValue = 0;\r
         } else if (!IsSelectable (NextMenuOption)) {\r
           //\r
           // Continue to go up until scroll to next page or the selectable option is found.\r
@@ -3467,9 +3501,15 @@ UiDisplayMenu (
       break;\r
 \r
     case CfUiPageUp:\r
+      //\r
+      // SkipValue means lines is skipped when show the top menu option.\r
+      //\r
       ControlFlag     = CfCheckSelection;\r
 \r
       ASSERT(NewPos != NULL);\r
+      //\r
+      // Already at the first menu option, so do nothing.\r
+      //\r
       if (NewPos->BackLink == &gMenuOption) {\r
         NewLine = FALSE;\r
         Repaint = FALSE;\r
@@ -3478,8 +3518,22 @@ UiDisplayMenu (
 \r
       NewLine   = TRUE;\r
       Repaint   = TRUE;\r
+\r
+      //\r
+      // SkipValue > (BottomRow - TopRow + 1) means current menu has more than one\r
+      // form of options to be show, so just update the SkipValue to show the next\r
+      // parts of options.\r
+      //\r
+      if (SkipValue > (INTN) (BottomRow - TopRow + 1)) {\r
+        SkipValue -= BottomRow - TopRow + 1;\r
+        break;\r
+      }\r
+\r
       Link      = TopOfScreen;\r
-      Index     = BottomRow;\r
+      //\r
+      // First minus the menu of the top screen, it's value is SkipValue.\r
+      //\r
+      Index     = (BottomRow + 1) - SkipValue;\r
       while ((Index >= TopRow) && (Link->BackLink != &gMenuOption)) {\r
         Link = Link->BackLink;\r
         PreviousMenuOption = MENU_OPTION_FROM_LINK (Link);\r
@@ -3487,13 +3541,13 @@ UiDisplayMenu (
           UpdateOptionSkipLines (Selection, PreviousMenuOption);\r
         }        \r
         if (Index < PreviousMenuOption->Skip) {\r
-          Index = 0;\r
           break;\r
         }\r
         Index = Index - PreviousMenuOption->Skip;\r
       }\r
       \r
       if ((Link->BackLink == &gMenuOption) && (Index >= TopRow)) {\r
+        SkipValue = 0;\r
         if (TopOfScreen == &gMenuOption) {\r
           TopOfScreen = gMenuOption.ForwardLink;\r
           NewPos      = gMenuOption.BackLink;\r
@@ -3512,10 +3566,13 @@ UiDisplayMenu (
           MoveToNextStatement (Selection, FALSE, &NewPos, BottomRow - TopRow);\r
         }\r
       } else {\r
-        if (Index + 1 < TopRow) {\r
+        if (Index >= TopRow) {\r
           //\r
-          // Back up the previous option.\r
+          // At here, only case "Index < PreviousMenuOption->Skip" can reach here.\r
           //\r
+          SkipValue = PreviousMenuOption->Skip - (Index - TopRow);\r
+        } else {\r
+          SkipValue = PreviousMenuOption->Skip - (TopRow - Index);\r
           Link = Link->ForwardLink;\r
         }\r
 \r
@@ -3546,6 +3603,9 @@ UiDisplayMenu (
       break;\r
 \r
     case CfUiPageDown:\r
+      //\r
+      // SkipValue means lines is skipped when show the top menu option.\r
+      //\r
       ControlFlag     = CfCheckSelection;\r
 \r
       ASSERT (NewPos != NULL);\r
@@ -3559,47 +3619,62 @@ UiDisplayMenu (
       Repaint = TRUE;\r
       Link    = TopOfScreen;\r
       NextMenuOption = MENU_OPTION_FROM_LINK (Link);\r
-      Index = TopRow;\r
-      while ((Index <= BottomRow) && (Link->ForwardLink != &gMenuOption)) {\r
-        Index = Index + NextMenuOption->Skip;\r
+      Index = TopRow + NextMenuOption->Skip - SkipValue;\r
+      //\r
+      // Count to the menu option which will show at the top of the next form.\r
+      //\r
+      while ((Index <= BottomRow + 1) && (Link->ForwardLink != &gMenuOption)) {\r
         Link           = Link->ForwardLink;\r
         NextMenuOption = MENU_OPTION_FROM_LINK (Link);\r
+        Index = Index + NextMenuOption->Skip;\r
       }\r
 \r
-      if ((Link->ForwardLink == &gMenuOption) && (Index <= BottomRow)) {\r
+      if ((Link->ForwardLink == &gMenuOption) && (Index <= BottomRow + 1)) {\r
         //\r
         // Finally we know that NewPos is the last MenuOption can be focused.\r
         //\r
         Repaint = FALSE;\r
         MoveToNextStatement (Selection, TRUE, &Link, Index - TopRow);\r
+        SkipValue = 0;\r
       } else {\r
-        if (Index - 1 > BottomRow) {\r
+        //\r
+        // Calculate the skip line for top of screen menu.\r
+        //\r
+        if (Link == TopOfScreen) {\r
           //\r
-          // Back up the previous option.\r
+          // The top of screen menu option occupies the entire form.\r
           //\r
-          Link = Link->BackLink;\r
+          SkipValue += BottomRow - TopRow + 1;\r
+        } else {\r
+          SkipValue = NextMenuOption->Skip - (Index - (BottomRow + 1));\r
         }\r
-        //\r
-        // There are more MenuOption needing scrolling down.\r
-        //\r
+\r
         TopOfScreen = Link;\r
         MenuOption = NULL;\r
         //\r
-        // Move to the option in Next page.\r
+        // Move to the Next selectable menu.\r
         //\r
         MoveToNextStatement (Selection, FALSE, &Link, BottomRow - TopRow);\r
       }\r
 \r
+      //\r
+      // Save the menu as the next highlight menu.\r
+      //\r
+      NewPos  = Link;\r
+\r
       //\r
       // If we encounter a Date/Time op-code set, rewind to the first op-code of the set.\r
       // Don't do this when we are already in the last page.\r
       //\r
-      NewPos  = Link;\r
       AdjustDateAndTimePosition (TRUE, &TopOfScreen);\r
       AdjustDateAndTimePosition (TRUE, &NewPos);\r
       break;\r
 \r
     case CfUiDown:\r
+      //\r
+      // SkipValue means lines is skipped when show the top menu option.\r
+      // NewPos  points to the menu which is highlighted now.\r
+      //\r
       ControlFlag = CfCheckSelection;\r
       //\r
       // Since the behavior of hitting the down arrow on a Date/Time op-code is intended\r
@@ -3618,8 +3693,14 @@ UiDisplayMenu (
         NewPos          = NewPos->ForwardLink;\r
 \r
         Difference      = 0;\r
+        //\r
+        // Current menu not at the bottom of the form.\r
+        //\r
         if (BottomRow >= MenuOption->Row + MenuOption->Skip) {\r
-          Difference    = MoveToNextStatement (Selection, FALSE, &NewPos, BottomRow - MenuOption->Row - MenuOption->Skip);\r
+          //\r
+          // Find the next selectable menu.\r
+          //\r
+          Difference = MoveToNextStatement (Selection, FALSE, &NewPos, BottomRow - MenuOption->Row - MenuOption->Skip);\r
           //\r
           // We hit the end of MenuOption that can be focused\r
           // so we simply scroll to the first page.\r
@@ -3637,7 +3718,8 @@ UiDisplayMenu (
             }\r
             NewPos        = gMenuOption.ForwardLink;\r
             MoveToNextStatement (Selection, FALSE, &NewPos, BottomRow - TopRow);\r
-    \r
+\r
+            SkipValue = 0;\r
             //\r
             // If we are at the end of the list and sitting on a Date/Time op, rewind to the head.\r
             //\r
@@ -3647,11 +3729,9 @@ UiDisplayMenu (
           }\r
         }\r
         NextMenuOption  = MENU_OPTION_FROM_LINK (NewPos);\r
-\r
-        //\r
-        // An option might be multi-line, so we need to reflect that data in the overall skip value\r
-        //\r
-        UpdateOptionSkipLines (Selection, NextMenuOption);\r
+        if (NextMenuOption->Row == 0) {\r
+          UpdateOptionSkipLines (Selection, NextMenuOption);\r
+        }\r
         DistanceValue  = Difference + NextMenuOption->Skip;\r
 \r
         Temp = MenuOption->Row + MenuOption->Skip + DistanceValue - 1;\r
@@ -3676,18 +3756,16 @@ UiDisplayMenu (
             //\r
             // If bottom op-code is more than one line or top op-code is more than one line\r
             //\r
-            if ((DistanceValue > 1) || (MenuOption->Skip > 1)) {\r
+            if ((DistanceValue > 1) || (SavedMenuOption->Skip > 1)) {\r
               //\r
               // Is the bottom op-code greater than or equal in size to the top op-code?\r
               //\r
-              if ((Temp - BottomRow) >= (SavedMenuOption->Skip - OldSkipValue)) {\r
+              if ((Temp - BottomRow) >= (SavedMenuOption->Skip - SkipValue)) {\r
                 //\r
                 // Skip the top op-code\r
                 //\r
                 TopOfScreen     = TopOfScreen->ForwardLink;\r
-                Difference      = (Temp - BottomRow) - (SavedMenuOption->Skip - OldSkipValue);\r
-\r
-                OldSkipValue    = Difference;\r
+                Difference      = (Temp - BottomRow) - (SavedMenuOption->Skip - SkipValue);\r
 \r
                 SavedMenuOption = MENU_OPTION_FROM_LINK (TopOfScreen);\r
 \r
@@ -3707,20 +3785,17 @@ UiDisplayMenu (
                 // SkipValue, set the skips to one less than what is required.\r
                 //\r
                 SkipValue = Difference - 1;\r
-\r
               } else {\r
                 //\r
                 // Since we will act on this op-code in the next routine, and increment the\r
                 // SkipValue, set the skips to one less than what is required.\r
                 //\r
-                SkipValue = OldSkipValue + (Temp - BottomRow) - 1;\r
+                SkipValue += (Temp - BottomRow) - 1;\r
               }\r
             } else {\r
-              if ((OldSkipValue + 1) == (INTN) SavedMenuOption->Skip) {\r
+              if ((SkipValue + 1) == (INTN) SavedMenuOption->Skip) {\r
                 TopOfScreen = TopOfScreen->ForwardLink;\r
                 break;\r
-              } else {\r
-                SkipValue = OldSkipValue;\r
               }\r
             }\r
             //\r
@@ -3742,7 +3817,6 @@ UiDisplayMenu (
           } while (SavedMenuOption->Skip == 0);\r
 \r
           Repaint       = TRUE;\r
-          OldSkipValue  = SkipValue;\r
         } else if (!IsSelectable (NextMenuOption)) {\r
           //\r
           // Continue to go down until scroll to next page or the selectable option is found.\r
@@ -3764,9 +3838,18 @@ UiDisplayMenu (
           Repaint     = TRUE;\r
           MenuOption  = NULL;\r
         } else {\r
+          //\r
+          // Need to remove the current highlight menu.\r
+          // MenuOption saved the last highlight menu info.\r
+          //\r
           MenuOption = MENU_OPTION_FROM_LINK (SavedListEntry);\r
         }\r
+\r
+        SkipValue     = 0;\r
         NewLine       = TRUE;\r
+        //\r
+        // Get the next highlight menu.\r
+        //\r
         NewPos        = gMenuOption.ForwardLink;\r
         MoveToNextStatement (Selection, FALSE, &NewPos, BottomRow - TopRow);\r
       }\r
@@ -3809,7 +3892,7 @@ UiDisplayMenu (
       // Reterieve default setting. After it. NV flag will be showed.\r
       //\r
       if ((HotKey->Action & BROWSER_ACTION_DEFAULT) == BROWSER_ACTION_DEFAULT) {\r
-        Status = ExtractDefault (Selection->FormSet, Selection->Form, HotKey->DefaultId, gBrowserSettingScope, GetDefaultForAll, NULL);\r
+        Status = ExtractDefault (Selection->FormSet, Selection->Form, HotKey->DefaultId, gBrowserSettingScope, GetDefaultForAll, NULL, FALSE);\r
         if (!EFI_ERROR (Status)) {\r
           Selection->Action = UI_ACTION_REFRESH_FORM;\r
           Selection->Statement = NULL;\r
@@ -3887,7 +3970,7 @@ UiDisplayMenu (
       //\r
       // Reset to default value for all forms in the whole system.\r
       //\r
-      Status = ExtractDefault (Selection->FormSet, NULL, DefaultId, FormSetLevel, GetDefaultForAll, NULL);\r
+      Status = ExtractDefault (Selection->FormSet, NULL, DefaultId, FormSetLevel, GetDefaultForAll, NULL, FALSE);\r
 \r
       if (!EFI_ERROR (Status)) {\r
         Selection->Action = UI_ACTION_REFRESH_FORM;\r