Title: [141234] trunk/Source/WebKit/chromium
- Revision
- 141234
- Author
- joc...@chromium.org
- Date
- 2013-01-30 01:37:54 -0800 (Wed, 30 Jan 2013)
Log Message
[chromium] WebConsoleMessage is missing LevelDebug (chromium bug 172416)
https://bugs.webkit.org/show_bug.cgi?id=108004
http://code.google.com/p/chromium/issues/detail?id=172416
console.debug triggers a NOTREACHED() assertation in Chromium. This
is because WebCore::MessageLevel contains 5 levels, including debug,
where WebConsoleMessage::Level is missing a "debug" level. Add a
WebConsoleMessage::LevelDebug so that it can get passed up to the
renderer even if it doesn't make use of that now.
Requires another patch to chromium itself to fix chromium bug 172416
but this is a prerequisite.
Also add an enum compile time check to AssertMatchingEnums.cpp,
Patch by Kevin Day <kevin...@gmail.com> on 2013-01-28
Reviewed by Jochen Eisinger.
* public/WebConsoleMessage.h:
* src/AssertMatchingEnums.cpp:
* src/WebFrameImpl.cpp:
(WebKit::WebFrameImpl::addMessageToConsole):
Modified Paths
Diff
Modified: trunk/Source/WebKit/chromium/ChangeLog (141233 => 141234)
--- trunk/Source/WebKit/chromium/ChangeLog 2013-01-30 09:28:19 UTC (rev 141233)
+++ trunk/Source/WebKit/chromium/ChangeLog 2013-01-30 09:37:54 UTC (rev 141234)
@@ -1,3 +1,28 @@
+2013-01-30 Jochen Eisinger <joc...@chromium.org>
+
+ [chromium] WebConsoleMessage is missing LevelDebug (chromium bug 172416)
+ https://bugs.webkit.org/show_bug.cgi?id=108004
+ http://code.google.com/p/chromium/issues/detail?id=172416
+
+ console.debug triggers a NOTREACHED() assertation in Chromium. This
+ is because WebCore::MessageLevel contains 5 levels, including debug,
+ where WebConsoleMessage::Level is missing a "debug" level. Add a
+ WebConsoleMessage::LevelDebug so that it can get passed up to the
+ renderer even if it doesn't make use of that now.
+
+ Requires another patch to chromium itself to fix chromium bug 172416
+ but this is a prerequisite.
+
+ Also add an enum compile time check to AssertMatchingEnums.cpp,
+
+ Patch by Kevin Day <kevin...@gmail.com> on 2013-01-28
+ Reviewed by Jochen Eisinger.
+
+ * public/WebConsoleMessage.h:
+ * src/AssertMatchingEnums.cpp:
+ * src/WebFrameImpl.cpp:
+ (WebKit::WebFrameImpl::addMessageToConsole):
+
2013-01-30 Xianzhu Wang <wangxian...@chromium.org>
[Chromium] Correct zoom for focused node when using compositor scaling
Modified: trunk/Source/WebKit/chromium/public/WebConsoleMessage.h (141233 => 141234)
--- trunk/Source/WebKit/chromium/public/WebConsoleMessage.h 2013-01-30 09:28:19 UTC (rev 141233)
+++ trunk/Source/WebKit/chromium/public/WebConsoleMessage.h 2013-01-30 09:37:54 UTC (rev 141234)
@@ -40,7 +40,8 @@
LevelTip,
LevelLog,
LevelWarning,
- LevelError
+ LevelError,
+ LevelDebug
};
Level level;
Modified: trunk/Source/WebKit/chromium/src/AssertMatchingEnums.cpp (141233 => 141234)
--- trunk/Source/WebKit/chromium/src/AssertMatchingEnums.cpp 2013-01-30 09:28:19 UTC (rev 141233)
+++ trunk/Source/WebKit/chromium/src/AssertMatchingEnums.cpp 2013-01-30 09:37:54 UTC (rev 141234)
@@ -635,3 +635,9 @@
COMPILE_ASSERT_MATCHING_ENUM(WebURLRequest::PriorityMedium, ResourceLoadPriorityMedium);
COMPILE_ASSERT_MATCHING_ENUM(WebURLRequest::PriorityHigh, ResourceLoadPriorityHigh);
COMPILE_ASSERT_MATCHING_ENUM(WebURLRequest::PriorityVeryHigh, ResourceLoadPriorityVeryHigh);
+
+COMPILE_ASSERT_MATCHING_ENUM(WebConsoleMessage::LevelTip, TipMessageLevel);
+COMPILE_ASSERT_MATCHING_ENUM(WebConsoleMessage::LevelLog, LogMessageLevel);
+COMPILE_ASSERT_MATCHING_ENUM(WebConsoleMessage::LevelWarning, WarningMessageLevel);
+COMPILE_ASSERT_MATCHING_ENUM(WebConsoleMessage::LevelError, ErrorMessageLevel);
+COMPILE_ASSERT_MATCHING_ENUM(WebConsoleMessage::LevelDebug, DebugMessageLevel);
Modified: trunk/Source/WebKit/chromium/src/WebFrameImpl.cpp (141233 => 141234)
--- trunk/Source/WebKit/chromium/src/WebFrameImpl.cpp 2013-01-30 09:28:19 UTC (rev 141233)
+++ trunk/Source/WebKit/chromium/src/WebFrameImpl.cpp 2013-01-30 09:37:54 UTC (rev 141234)
@@ -837,6 +837,9 @@
case WebConsoleMessage::LevelError:
webCoreMessageLevel = ErrorMessageLevel;
break;
+ case WebConsoleMessage::LevelDebug:
+ webCoreMessageLevel = DebugMessageLevel;
+ break;
default:
ASSERT_NOT_REACHED();
return;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
