Title: [132643] trunk/Source/WebKit2
- Revision
- 132643
- Author
- [email protected]
- Date
- 2012-10-26 06:17:51 -0700 (Fri, 26 Oct 2012)
Log Message
[EFL][WK2] Initialize Ewk_Context class members in initialization list
https://bugs.webkit.org/show_bug.cgi?id=100503
Patch by Christophe Dumez <[email protected]> on 2012-10-26
Reviewed by Kenneth Rohde Christiansen.
Move Ewk_Context class members initialization from the
constructor body to the initialization list for
efficiency.
* UIProcess/API/efl/ewk_context.cpp:
(Ewk_Context::Ewk_Context):
Modified Paths
Diff
Modified: trunk/Source/WebKit2/ChangeLog (132642 => 132643)
--- trunk/Source/WebKit2/ChangeLog 2012-10-26 13:14:50 UTC (rev 132642)
+++ trunk/Source/WebKit2/ChangeLog 2012-10-26 13:17:51 UTC (rev 132643)
@@ -1,3 +1,17 @@
+2012-10-26 Christophe Dumez <[email protected]>
+
+ [EFL][WK2] Initialize Ewk_Context class members in initialization list
+ https://bugs.webkit.org/show_bug.cgi?id=100503
+
+ Reviewed by Kenneth Rohde Christiansen.
+
+ Move Ewk_Context class members initialization from the
+ constructor body to the initialization list for
+ efficiency.
+
+ * UIProcess/API/efl/ewk_context.cpp:
+ (Ewk_Context::Ewk_Context):
+
2012-10-26 Jocelyn Turcotte <[email protected]>
LayerTreeCoordinator: Make sure that dependent layers are deleted before their directly composited image
Modified: trunk/Source/WebKit2/UIProcess/API/efl/ewk_context.cpp (132642 => 132643)
--- trunk/Source/WebKit2/UIProcess/API/efl/ewk_context.cpp 2012-10-26 13:14:50 UTC (rev 132642)
+++ trunk/Source/WebKit2/UIProcess/API/efl/ewk_context.cpp 2012-10-26 13:17:51 UTC (rev 132643)
@@ -62,22 +62,21 @@
Ewk_Context::Ewk_Context(WKContextRef context)
: m_context(context)
- , m_historyClient(ContextHistoryClientEfl::create(context))
-{
- ContextMap::AddResult result = contextMap().add(context, this);
- ASSERT_UNUSED(result, result.isNewEntry);
-
#if ENABLE(BATTERY_STATUS)
- m_batteryProvider = BatteryProvider::create(context);
+ , m_batteryProvider(BatteryProvider::create(context))
#endif
-
#if ENABLE(NETWORK_INFO)
- m_networkInfoProvider = NetworkInfoProvider::create(context);
+ , m_networkInfoProvider(NetworkInfoProvider::create(context))
#endif
-
#if ENABLE(VIBRATION)
- m_vibrationProvider = VibrationProvider::create(context);
+ , m_vibrationProvider(VibrationProvider::create(context))
#endif
+ , m_downloadManager(DownloadManagerEfl::create(this))
+ , m_requestManagerClient(RequestManagerClientEfl::create(this))
+ , m_historyClient(ContextHistoryClientEfl::create(context))
+{
+ ContextMap::AddResult result = contextMap().add(context, this);
+ ASSERT_UNUSED(result, result.isNewEntry);
#if ENABLE(MEMORY_SAMPLER)
static bool initializeMemorySampler = false;
@@ -97,10 +96,6 @@
ewk_settings_spell_checking_languages_set(0);
}
#endif
-
- // Initialize WKContext clients.
- m_downloadManager = DownloadManagerEfl::create(this);
- m_requestManagerClient = RequestManagerClientEfl::create(this);
}
Ewk_Context::~Ewk_Context()
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes
