fixup! [M47_2526] Chromium upversion to m47_2526 branch

Use hasNonEmptyLayoutSize instead of hasNonEmptyBoundingBox. See
https://codereview.chromium.org/1226243002.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14509

Reviewed by: a1.gomes, sns.park

Change-Id: I28aabd097d70c7e0956b44642409916175c35f9c
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>

diff --git a/tizen_src/ewk/efl_integration/renderer/render_frame_observer_efl.cc b/tizen_src/ewk/efl_integration/renderer/render_frame_observer_efl.cc
index 534a47f41225739559a95d29366e69de8380d31f..b2dd3dff7fe219ab33a95b8655e20a6ec7739427 100644 (file)
--- a/tizen_src/ewk/efl_integration/renderer/render_frame_observer_efl.cc
+++ b/tizen_src/ewk/efl_integration/renderer/render_frame_observer_efl.cc
@@ -140,19 +140,15 @@ void RenderFrameObserverEfl::OnRequestSelectCollectionInformation() {
   WebElementCollection select_elements = document.getElementsByHTMLTagName("select");
   int count = 0;
   int index = 0;
-#if !defined(EWK_BRINGUP)
-// [M47_2526] The hasNonEmptyBoundingBox was removed
-//            FIXME: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=14509
   for (WebElement e = select_elements.firstItem();
       !e.isNull(); e = select_elements.nextItem()) {
     // take only visible elements into account
-    if (e.hasNonEmptyBoundingBox()) {
+    if (e.hasNonEmptyLayoutSize()) {
       if (e == focused_element)
         index = count;
       ++count;
     }
   }
-#endif
 
   if (count) {
     bool prev_status = index > 0;