Title: [140326] trunk/Source/WebKit/qt
- Revision
- 140326
- Author
- [email protected]
- Date
- 2013-01-21 05:40:06 -0800 (Mon, 21 Jan 2013)
Log Message
[Qt] Update the documentation about the storage of inspector settings
https://bugs.webkit.org/show_bug.cgi?id=106777
Reviewed by Allan Sandfeld Jensen.
Since r76770, QSettings isn't queried anymore to persist inspector
settings like whether or not the debugger is enabled by default.
It still seems to be used for the default attach height.
Update the documentation accordingly.
* WidgetApi/qwebinspector.cpp:
Modified Paths
Diff
Modified: trunk/Source/WebKit/qt/ChangeLog (140325 => 140326)
--- trunk/Source/WebKit/qt/ChangeLog 2013-01-21 13:34:11 UTC (rev 140325)
+++ trunk/Source/WebKit/qt/ChangeLog 2013-01-21 13:40:06 UTC (rev 140326)
@@ -1,3 +1,18 @@
+2013-01-21 Jocelyn Turcotte <[email protected]>
+
+ [Qt] Update the documentation about the storage of inspector settings
+ https://bugs.webkit.org/show_bug.cgi?id=106777
+
+ Reviewed by Allan Sandfeld Jensen.
+
+ Since r76770, QSettings isn't queried anymore to persist inspector
+ settings like whether or not the debugger is enabled by default.
+ It still seems to be used for the default attach height.
+
+ Update the documentation accordingly.
+
+ * WidgetApi/qwebinspector.cpp:
+
2013-01-21 Allan Sandfeld Jensen <[email protected]>
[Qt] Crash in gmail on enabling desktop notifications
Modified: trunk/Source/WebKit/qt/WidgetApi/qwebinspector.cpp (140325 => 140326)
--- trunk/Source/WebKit/qt/WidgetApi/qwebinspector.cpp 2013-01-21 13:34:11 UTC (rev 140325)
+++ trunk/Source/WebKit/qt/WidgetApi/qwebinspector.cpp 2013-01-21 13:40:06 UTC (rev 140326)
@@ -66,11 +66,13 @@
The inspector allows the user to configure some options through its
user interface (e.g. the resource tracking "Always enable" option).
- These settings will be persisted automatically by QtWebKit only if
- your application previously called QCoreApplication::setOrganizationName()
- and QCoreApplication::setApplicationName().
- See QSettings's default constructor documentation for an explanation
- of why this is necessary.
+ The inspector UI is itself a web page and is using HTML local storage
+ to persist those settings.
+ Since the internal QWebPage used by the inspector isn't exposed in the API,
+ the only way to enable those settings to be persisted is currently to enable
+ local storage globally through QWebSettings::globalSettings().
+
+ \sa QWebSettings::localStoragePath(), QWebSettings::LocalStorageEnabled
*/
/*!
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes
