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

Side by Side Diff: content/browser/browser_plugin/browser_plugin_embedder.h

Issue 2755723002: Fixed an IME bug due to a race in TextInputStateChanged
Patch Set: Using BrowserPlugingEmbedder for determinning focused guests Created 3 years, 9 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 unified diff | Download patch
« no previous file with comments | « no previous file | content/browser/browser_plugin/browser_plugin_embedder.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // A BrowserPluginEmbedder handles messages coming from a BrowserPlugin's 5 // A BrowserPluginEmbedder handles messages coming from a BrowserPlugin's
6 // embedder that are not directed at any particular existing guest process. 6 // embedder that are not directed at any particular existing guest process.
7 // In the beginning, when a BrowserPlugin instance in the embedder renderer 7 // In the beginning, when a BrowserPlugin instance in the embedder renderer
8 // process requests an initial navigation, the WebContents for that renderer 8 // process requests an initial navigation, the WebContents for that renderer
9 // renderer creates a BrowserPluginEmbedder for itself. The 9 // renderer creates a BrowserPluginEmbedder for itself. The
10 // BrowserPluginEmbedder, in turn, forwards the requests to a 10 // BrowserPluginEmbedder, in turn, forwards the requests to a
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
83 83
84 // Returns the "full page" guest if there is one. That is, if there is a 84 // Returns the "full page" guest if there is one. That is, if there is a
85 // single BrowserPlugin in the embedder which takes up the full page, then it 85 // single BrowserPlugin in the embedder which takes up the full page, then it
86 // is returned. 86 // is returned.
87 BrowserPluginGuest* GetFullPageGuest(); 87 BrowserPluginGuest* GetFullPageGuest();
88 88
89 // Polls all guests for this web contents and returns true if any of them 89 // Polls all guests for this web contents and returns true if any of them
90 // were audible recently. 90 // were audible recently.
91 bool WereAnyGuestsRecentlyAudible(); 91 bool WereAnyGuestsRecentlyAudible();
92 92
93 // TODO(ekaramad): This is solely used for BrowserPlugin. Remove when issue
94 // https://crbug.com/533069 is fixed.
95 // Returns true if there are any focused guests inside the embedder.
96 bool AreAnyGuestsFocused();
97
93 private: 98 private:
94 explicit BrowserPluginEmbedder(WebContentsImpl* web_contents); 99 explicit BrowserPluginEmbedder(WebContentsImpl* web_contents);
95 100
96 BrowserPluginGuestManager* GetBrowserPluginGuestManager() const; 101 BrowserPluginGuestManager* GetBrowserPluginGuestManager() const;
97 102
98 void ClearGuestDragStateIfApplicable(); 103 void ClearGuestDragStateIfApplicable();
99 104
100 static bool DidSendScreenRectsCallback(WebContents* guest_web_contents); 105 static bool DidSendScreenRectsCallback(WebContents* guest_web_contents);
101 106
102 // Notifies a guest that the embedder's screen info has changed. 107 // Notifies a guest that the embedder's screen info has changed.
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
137 bool guest_drag_ending_; 142 bool guest_drag_ending_;
138 143
139 base::WeakPtrFactory<BrowserPluginEmbedder> weak_ptr_factory_; 144 base::WeakPtrFactory<BrowserPluginEmbedder> weak_ptr_factory_;
140 145
141 DISALLOW_COPY_AND_ASSIGN(BrowserPluginEmbedder); 146 DISALLOW_COPY_AND_ASSIGN(BrowserPluginEmbedder);
142 }; 147 };
143 148
144 } // namespace content 149 } // namespace content
145 150
146 #endif // CONTENT_BROWSER_BROWSER_PLUGIN_BROWSER_PLUGIN_EMBEDDER_H_ 151 #endif // CONTENT_BROWSER_BROWSER_PLUGIN_BROWSER_PLUGIN_EMBEDDER_H_
OLDNEW
« no previous file with comments | « no previous file | content/browser/browser_plugin/browser_plugin_embedder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698