summaryrefslogtreecommitdiffstats
path: root/MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c
diff options
context:
space:
mode:
authorEric Dong <eric.dong@intel.com>2014-10-21 05:50:06 +0000
committerydong10 <ydong10@Edk2>2014-10-21 05:50:06 +0000
commitca2be854872373f18fc1cdf5277973fcd9179e7a (patch)
treeb5a1174aac8348ba05100bc02ec4511979353ce8 /MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c
parent79e1ffbdd4cb6ea4d1ec8c25a1f6c905a57458f0 (diff)
downloadedk2-ca2be854872373f18fc1cdf5277973fcd9179e7a.tar.gz
edk2-ca2be854872373f18fc1cdf5277973fcd9179e7a.tar.bz2
edk2-ca2be854872373f18fc1cdf5277973fcd9179e7a.zip
Check the question value before call CHANGED. Only when question value has been changed, browser will call CHANGED callback type.
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Eric Dong <eric.dong@intel.com> Reviewed-by: Liming Gao <liming.gao@intel.com> git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@16222 6f19259b-4bc3-4df7-8a09-765794883524
Diffstat (limited to 'MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c')
-rw-r--r--MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c7
1 files changed, 6 insertions, 1 deletions
diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c b/MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c
index f0617f81b3..bfb8afc140 100644
--- a/MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c
+++ b/MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c
@@ -2426,7 +2426,12 @@ SetupBrowser (
//
IsQuestionValueChanged(gCurrentSelection->FormSet, gCurrentSelection->Form, Statement, GetSetValueWithBuffer);
- if (!EFI_ERROR (Status) && Statement->Operand != EFI_IFR_REF_OP) {
+ if (!EFI_ERROR (Status) &&
+ (Statement->Operand != EFI_IFR_REF_OP) &&
+ ((Statement->Storage == NULL) || (Statement->Storage != NULL && Statement->ValueChanged))) {
+ //
+ // Only question value has been changed, browser will trig CHANGED callback.
+ //
ProcessCallBackFunction(Selection, Selection->FormSet, Selection->Form, Statement, EFI_BROWSER_ACTION_CHANGED, FALSE);
}
} else {