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

Unified Diff: chrome/browser/views/find_bar_host.cc

Issue 438010: Moving two tests to FLAKY tests. (Closed)
Patch Set: comment fix Created 11 years, 1 month 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 | « chrome/browser/ssl/ssl_browser_tests.cc ('k') | chrome/browser/views/find_bar_host_browsertest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/views/find_bar_host.cc
diff --git a/chrome/browser/views/find_bar_host.cc b/chrome/browser/views/find_bar_host.cc
index 9b2cda555c2d2e81b5d53da6b8017f4b24e8dea8..dd8622a4abb24db074aa531b84b1a3ea34d4b373 100644
--- a/chrome/browser/views/find_bar_host.cc
+++ b/chrome/browser/views/find_bar_host.cc
@@ -117,6 +117,8 @@ bool FindBarHost::GetFindBarWindowInfo(gfx::Point* position,
#else
false) {
// TODO(sky): figure out linux side.
+ // This is tricky due to asynchronous nature of x11.
+ // See bug http://crbug.com/28629.
#endif
if (position)
*position = gfx::Point();
« no previous file with comments | « chrome/browser/ssl/ssl_browser_tests.cc ('k') | chrome/browser/views/find_bar_host_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698