Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(96)

Unified Diff: ui/display/manager/forwarding_display_delegate.h

Issue 2835933005: Make ForwardingDisplayDelegate initially sync. (Closed)
Patch Set: Add comment explaining sync usage. Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « services/ui/display/screen_manager_forwarding.cc ('k') | ui/display/manager/forwarding_display_delegate.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/display/manager/forwarding_display_delegate.h
diff --git a/ui/display/manager/forwarding_display_delegate.h b/ui/display/manager/forwarding_display_delegate.h
index 4e548754d7c72e15d80c7f3aee1e8d0ea59a63ae..17cded9103b466bf575071adb96f51b26029d1f1 100644
--- a/ui/display/manager/forwarding_display_delegate.h
+++ b/ui/display/manager/forwarding_display_delegate.h
@@ -69,11 +69,18 @@ class DISPLAY_MANAGER_EXPORT ForwardingDisplayDelegate
void OnConfigurationChanged() override;
private:
- // Stores display snapshots and forards them to |callback|.
+ // Stores display snapshots and forwards pointers to |callback|.
void StoreAndForwardDisplays(
const GetDisplaysCallback& callback,
std::vector<std::unique_ptr<DisplaySnapshotMojo>> snapshots);
+ // Forwards display snapshot pointers to |callback|.
+ void ForwardDisplays(const GetDisplaysCallback& callback);
+
+ // True if we should use |delegate_|. This will be false if synchronous
+ // GetDisplays() and Configure() are required.
+ bool use_delegate_ = false;
+
mojom::NativeDisplayDelegatePtr delegate_;
mojo::Binding<mojom::NativeDisplayObserver> binding_;
« no previous file with comments | « services/ui/display/screen_manager_forwarding.cc ('k') | ui/display/manager/forwarding_display_delegate.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698