Remove use of WebNode/WebElement in translate_helper

third_party/WebKit/public/web/WebLanguageDetectionDetails.h

+// Copyright 2016 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.
+
+#ifndef WebLanguageDetectionDetails_h
+#define WebLanguageDetectionDetails_h
+
+#include "public/platform/WebString.h"
+#include "public/platform/WebURL.h"
+#include "public/web/WebDocument.h"

[jbroman, 2016/12/20 16:43:51]

nit: Technically you just need a forward declaration here. You can remove this
include and put "class WebDocument;" inside the Blink namespace.

[adithyas, 2017/01/09 15:40:02]

Done.

+
+namespace blink {
+
+struct WebLanguageDetectionDetails {
+  WebURL url;
+  WebString contentLanguage;
+  WebString htmlLanguage;
+  bool hasNoTranslateMeta;

[jbroman, 2016/12/20 16:43:51]

nit: Rather than writing an explicit constructor, you can just write "bool
hasNoTranslateMeta = false;" here.

[adithyas, 2017/01/09 15:40:02]

Removed the constructor.

+
+  WebLanguageDetectionDetails() : hasNoTranslateMeta(false) {}
+
+  BLINK_EXPORT static WebLanguageDetectionDetails
+  collectLanguageDetectionDetails(const WebDocument&);
+};
+
+}  // namespace blink
+
+#endif  // WebLanguageDetectionDetails_h