Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
jamescook@ -> c/b/extensions no need to review other dirs
xdai@ -> components/app_restore/full_restore_read_and_save_unittest.cc no need to review other dirs
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Code-Review | +1 |
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Code-Review | +1 |
Code-Review | +1 |
Nice refactoring. Highly recommend a gemini-cli extension developed by our own @ju...@google.com for finding out the exact definition/declaration/call sites of a function. In our case here, the function is `aura::test::CreateTestWindow()`.
Per this extension, gemini-cli reported the following files to be further refactored to complete your preparation effort. For your reference, the remaining files are:
```
- ash/wm/tablet_mode/tablet_mode_window_manager_unittest.cc
- ash/wm/window_dimmer_unittest.cc
- ash/wm/window_manager_unittest.cc
- ash/wm/window_modality_controller_unittest.cc
- ash/wm/workspace_controller_unittest.cc
- chrome/browser/apps/app_service/metrics/app_platform_metrics_service_unittest.cc
- chrome/browser/ash/arc/instance_throttle/arc_active_window_throttle_observer_unittest.cc
- chrome/browser/ash/arc/instance_throttle/arc_pip_window_throttle_observer_unittest.cc
- chrome/browser/ash/crostini/throttle/crostini_active_window_throttle_observer_unittest.cc
- chrome/browser/ash/exo/chrome_data_exchange_delegate_unittest.cc
- chrome/browser/ash/exo/chrome_security_delegate_unittest.cc
- chrome/browser/performance_manager/policies/working_set_trimmer_policy_arcvm_unittest.cc
- chrome/browser/ui/views/tabs/dragging/tab_drag_controller_interactive_uitest.cc
- chromeos/ash/experiences/arc/ime/arc_ime_service_unittest.cc
- content/browser/web_contents/web_contents_view_aura_unittest.cc
- ui/aura/gestures/gesture_recognizer_unittest.cc
- ui/aura/test/aura_test_base.cc
- ui/aura/window_event_dispatcher_unittest.cc
- ui/aura/window_occlusion_tracker_unittest.cc
- ui/aura/window_unittest.cc
- ui/snapshot/snapshot_aura_unittest.cc
- ui/wm/core/compound_event_filter_unittest.cc
- ui/wm/core/focus_controller_unittest.cc
- ui/wm/core/ime_util_chromeos_unittest.cc
- ui/wm/core/shadow_controller_unittest.cc
- ui/wm/core/transient_window_manager_unittest.cc
- ui/wm/core/transient_window_stacking_client_unittest.cc
```
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Code-Review | +1 |
components/app_restore/full_restore_read_and_save_unittest.cc LGTM
Thanks, the list in this CL is indeed incomplete, due to the issue with gemini and I had to stop in the middle. The most of files above already have .bounds, so I think I can just finish the rest using standard code search that Gemini uses. (which also search xref)
That's being said, the existing one has several issues, such as it, *) couldn't handle overloading well, *) it sometimes generate bad regexp, *) generates wrong edit command. Do you think it can address these issues?
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Commit-Queue | +2 |
I'll defer this question to @ju...@google.com. Closing this thread for now.
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Commit-Queue | +2 |
Specify bounds when creating WindowBuilderParams.
This {100, 100} is the current default avlue, and I'm planning to
change this to {}. This is a preparation to change the default
value of `bounds` to empty
(generated by gemini-cli)
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |