Introduce NavigatorID IDL interface

Source/core/page/NavigatorBase.h

Index: Source/core/page/NavigatorBase.h
diff --git a/Source/core/page/NavigatorBase.h b/Source/core/page/NavigatorBase.h
index dcc9d067c68ebd5c3982bf4297d82f71c95b92ba..f6772abe875e5e81b35aba0c8443c3ab00ee13e7 100644
--- a/Source/core/page/NavigatorBase.h
+++ b/Source/core/page/NavigatorBase.h
@@ -32,13 +32,16 @@ namespace WebCore {
 
     class NavigatorBase {
     public:
-        String appName() const;
-        String appVersion() const;
         virtual String userAgent() const = 0;
-        String platform() const;
 
+        // FIXME: The following should be moved to NavigatorID and

[do-not-use, 2013/08/20 11:05:06]

Planning to address this in a follow up patch. Should not be controversial.

+        // exposed on WorkerNavigator as well.
         String appCodeName() const;
         String product() const;
+
+        // FIXME: The following is not part of the specification and is only

[do-not-use, 2013/08/20 11:05:06]

Planning to address this in a follow-up patch. My personal preference would be
to move this to Navigator and keep exposing this on Navigator.idl only since
this does not seem to be standard. However, advice is welcome.

[haraken, 2013/08/20 12:52:52]

+1 to moving the methods to Navigator.idl only.

[arv (Not doing code reviews), 2013/08/20 13:40:44]

+1 and add deprecation warnings on these as well. IE11 did not add them so these
should be safe to remove on the open web.

+        // exposed on Navigator. It should be moved to Navigator (or exposed
+        // on WorkerNavigator as well).
         String productSub() const;
         String vendor() const;
         String vendorSub() const;