Handle drop cap letters

java/org/chromium/distiller/webdocument/ElementAction.java

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 package org.chromium.distiller.webdocument;
 
 import org.chromium.distiller.DomUtil;
 import org.chromium.distiller.labels.DefaultLabels;
 
 import com.google.gwt.core.client.JavaScriptObject;
 import com.google.gwt.core.client.JsArrayString;
 import com.google.gwt.dom.client.Element;
 import com.google.gwt.dom.client.Style;
 import com.google.gwt.regexp.shared.RegExp;
 
 public class ElementAction {
     public boolean changesTagLevel = false;
     public boolean flush = false;
     public boolean isAnchor = false;
     public JsArrayString labels = JavaScriptObject.createArray().<JsArrayString>cast();
 
     private static final RegExp REG_COMMENT = RegExp.compile("\\bcomments?\\b");
     private static final int MAX_CLASS_COUNT = 5;
 
     public static ElementAction getForElement(Element element) {
         Style style = DomUtil.getComputedStyle(element);
         ElementAction action = new ElementAction();
+        String tagName = element.getTagName();
         switch (style.getDisplay()) {
             case "inline":
                 break;
             case "inline-block":
             case "inline-flex":
                 action.changesTagLevel = true;
                 break;
+            case "block":

[mdjones, 2016/03/14 16:58:27]

Please add a comment outlining that the float property changes how the element
is displayed. 

[wychen, 2016/03/15 20:47:59]

Done.

+                // Special casing for drop cap letter with "float".
+                // Ref: http://crbug.com/593128
+                if (!"none".equals(style.getProperty("float")) &&
+                    "SPAN".equals(tagName)) {
+                    break;
+                }
+                // Intentional fall through.
             // See http://www.w3.org/TR/CSS2/tables.html#table-display
             // and http://www.w3.org/TR/css-flexbox-1/#flex-containers
             // The default case includes the following display types:
-            // block
             // list-item
             // inline-table
             // table-row
             // table-row-group
             // table-header-group
             // table-footer-group
             // table-column
             // table-column-group
             // table-cell
             // table-caption
             // flex
             default:
                 action.flush = true;
                 action.changesTagLevel = true;
                 break;
         }
 
-        String tagName = element.getTagName();
         if (!"HTML".equals(tagName) && !"BODY".equals(tagName)) {
             String className = element.getAttribute("class");
             int classCount = DomUtil.getClassList(element).length();
             String id = element.getAttribute("id");
             if ((REG_COMMENT.test(className) || REG_COMMENT.test(id)) &&
                     classCount <= MAX_CLASS_COUNT) {
                 action.labels.push(DefaultLabels.STRICTLY_NOT_CONTENT);
             }
 
             switch (tagName) {
@@ skipping 24 matching lines @@
                         action.isAnchor = true;
                     }
                     break;
             }
         }
         return action;
     }
 
     private ElementAction() {}
 }