Ignore the "Edit" links in wiki pages

java/org/chromium/distiller/webdocument/DomConverter.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.DomWalker;
-import org.chromium.distiller.JavaScript;
 import org.chromium.distiller.LogUtil;
 import org.chromium.distiller.TableClassifier;
 
+import com.google.gwt.dom.client.AnchorElement;
 import com.google.gwt.dom.client.Element;
 import com.google.gwt.dom.client.Node;
 import com.google.gwt.dom.client.Style;
 import com.google.gwt.dom.client.TableElement;
 import com.google.gwt.dom.client.Text;
 import org.chromium.distiller.extractors.embeds.EmbedExtractor;
 import org.chromium.distiller.extractors.embeds.ImageExtractor;
 import org.chromium.distiller.extractors.embeds.TwitterExtractor;
 import org.chromium.distiller.extractors.embeds.VimeoExtractor;
 import org.chromium.distiller.extractors.embeds.YouTubeExtractor;
@@ skipping 125 matching lines @@
             // See crbug.com/692553, crbug.com/696556, and crbug.com/674557
             return false;
         }
 
         // Create a placeholder for the elements we want to preserve.
         if (WebTag.canBeNested(e.getTagName())) {
             builder.tag(new WebTag(e.getTagName(), WebTag.TagType.START));
         }
 
         switch (e.getTagName()) {
+            case "A":
+                String editPattern = "action=edit&section=";

[mdjones, 2017/03/14 15:59:50]

Is "action=edit" used for more than just editing? How necessary is the
"&section="?

[wychen, 2017/03/14 21:48:54]

For "redlinks" (links to entries yet to be created), the href pattern is
"action=edit&redlink=". I've added a comment.

I didn't know this either until running eval.

+                boolean isEdit = AnchorElement.as(e).getHref().indexOf(editPattern) != -1;
+                if (isEdit) {
+                    // Skip "edit section" on mediawiki.
+                    // See crbug.com/647667.
+                    return false;
+                }
+                break;
+            case "SPAN":
+                if (className.equals("mw-editsection")) {
+                    // Skip "[edit]" on mediawiki desktop version.
+                    // See crbug.com/647667.
+                    return false;
+                }
+                break;
             case "BR":
                 builder.lineBreak(e);
                 return false;
             // Skip data tables, keep track of them to be extracted by RelevantElementsFinder
             // later.
             case "TABLE":
                 TableClassifier.Type type = TableClassifier.table(TableElement.as(e));
                 logTableInfo(e, type);
                 if (type == TableClassifier.Type.DATA) {
                     builder.dataTable(e);
@@ skipping 59 matching lines @@
         Element parent = e.getParentElement();
         LogUtil.logToConsole("TABLE: " + type +
                 ", id=" + e.getId() +
                 ", class=" + e.getAttribute("class") +
                 ", parent=[" + parent.getTagName() +
                 ", id=" + parent.getId() +
                 ", class=" + parent.getAttribute("class") +
                 "]");
     }
 }