The content='...' parsing error is not a viewport error, so keep it separate

Source/core/dom/Document.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) 2008, 2009, 2011, 2012 Google Inc. All rights reserved.
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) Research In Motion Limited 2010-2011. All rights reserved.
@@ skipping 3012 matching lines @@
             i++;
         }
         valueEnd = i;
 
         ASSERT_WITH_SECURITY_IMPLICATION(i <= length);
 
         String keyString = buffer.substring(keyBegin, keyEnd - keyBegin);
         String valueString = buffer.substring(valueBegin, valueEnd - valueBegin);
         callback(keyString, valueString, this, data);
     }
-    if (error)
-        reportViewportWarning(this, InvalidKeyValuePairSeparatorError, String(), String());
+    if (error) {
+        String message = "Note that ';' is not a key-value pair separator. The list should be comma-separated.";

[Mike West, 2013/09/06 12:17:47]

Since you're changing this anyway, I'd suggest making it clear that the meta tag
is the problem. Something like "Error parsing a meta element's content: ';' is
not a key-value pair separator, etc.".

I think we should do that for the other messages as well, now that I look at
this. But not in this patch. :)

+        addConsoleMessage(OtherMessageSource, WarningMessageLevel, message);

[Mike West, 2013/09/06 12:17:47]

I believe this should be 'RenderingMessageSource'.

+    }
 }
 
 void Document::processViewport(const String& features, ViewportArguments::Type origin)
 {
     ASSERT(!features.isNull());
 
     // We are adding viewport properties from a legacy meta tag.
     // The different meta tags have different priorities based on the type regardless
     // of which order they appear in the DOM. The priority is given by the
     // ViewportArguments::Type enum. If we process viewport properties with a lower
@@ skipping 2390 matching lines @@
 {
     return DocumentLifecycleNotifier::create(this);
 }
 
 DocumentLifecycleNotifier* Document::lifecycleNotifier()
 {
     return static_cast<DocumentLifecycleNotifier*>(ScriptExecutionContext::lifecycleNotifier());
 }
 
 } // namespace WebCore