depthcharge/ui: Remove obsolete TODO comments
For b/147424699, since CL:2386896 has already enabled buffered UI, there
is no more flickering issues when moving the cursor.
For b/163301076, the disabled button feature has been implemented in
CL:2432169, so remove the obsolete TODO comment.
BUG=b:147424699, b:163301076
TEST=none
BRANCH=none
Change-Id: Ib23355d60603a339814c64a95be27434364b26fb
Signed-off-by: Yu-Ping Wu <yupingso@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/depthcharge/+/2521069
Reviewed-by: Hsuan Ting Chen <roccochen@chromium.org>
Commit-Queue: Hsuan Ting Chen <roccochen@chromium.org>
diff --git a/src/vboot/ui/layout.c b/src/vboot/ui/layout.c
index 30551b5..53db8b3 100644
--- a/src/vboot/ui/layout.c
+++ b/src/vboot/ui/layout.c
@@ -667,10 +667,6 @@
continue;
if (VB2_GET_BIT(state->hidden_item_mask, i))
continue;
- /*
- * TODO(b/147424699): No need to redraw every button when
- * navigating between menu.
- */
clear_help = prev_state &&
prev_state->selected_item == i &&
VB2_GET_BIT(prev_state->disabled_item_mask, i);
@@ -729,7 +725,6 @@
/* Clear everything above the footer for new screen. */
const int32_t box_height = UI_SCALE - UI_MARGIN_BOTTOM -
UI_FOOTER_HEIGHT;
- /* TODO(b/147424699): Do not clear and redraw language header */
VB2_TRY(ui_draw_box(0, 0, UI_SCALE, box_height,
&ui_color_bg, 0));
}
diff --git a/src/vboot/ui/screens.c b/src/vboot/ui/screens.c
index aa3f004..5f48c9b 100644
--- a/src/vboot/ui/screens.c
+++ b/src/vboot/ui/screens.c
@@ -89,10 +89,6 @@
return batt_pct < 10;
}
-/*
- * TODO(b/163301076): Reconsider the functionalities of page up/down buttons
- * when reaching the start/end of the log.
- */
static vb2_error_t draw_log_desc(const struct ui_state *state,
const struct ui_state *prev_state,
int32_t *y)