Changeset 58259 in webkit

Timestamp:

Apr 26, 2010 9:56:42 AM (14 years ago)

Author:

evan@chromium.org

Message:

2010-04-26 Roman Gershman <​romange@google.com>

Reviewed by Adam Barth.

[Chromium] Font size in suggestions popup menu should be correlated with the font size of its text field.

​https://bugs.webkit.org/show_bug.cgi?id=37977

• src/SuggestionsPopupMenuClient.cpp: (WebKit::SuggestionsPopupMenuClient::initialize):

Location:

trunk/WebKit/chromium

Files:

• ChangeLog (modified) (1 diff)
• src/SuggestionsPopupMenuClient.cpp (modified) (1 diff)

trunk/WebKit/chromium/ChangeLog

@@ +1 @@
+2010-04-26  Roman Gershman  <romange@google.com>
+
+        Reviewed by Adam Barth.
+
+        [Chromium] Font size in suggestions popup menu should be correlated with the font size of its text field.
+
+        https://bugs.webkit.org/show_bug.cgi?id=37977
+
+        * src/SuggestionsPopupMenuClient.cpp:
+        (WebKit::SuggestionsPopupMenuClient::initialize):
+
 2010-04-26  Pavel Feldman  <pfeldman@chromium.org>
 

trunk/WebKit/chromium/src/SuggestionsPopupMenuClient.cpp

@@ -157 +157 @@
     RenderTheme::defaultTheme()->systemFont(CSSValueWebkitControl,
                                             fontDescription);
+    RenderStyle* style = m_textField->computedStyle();
+    fontDescription.setComputedSize(style->fontDescription().computedSize());
 
-    // Use a smaller font size to match IE/Firefox.
-    // FIXME: http://crbug.com/7376 use the system size instead of a
-    //        fixed font size value.
-    fontDescription.setComputedSize(12.0);
     Font font(fontDescription, 0, 0);
     font.update(textField->document()->styleSelector()->fontSelector());