Improve trace logs

sailfishos · Dec 5, 2018 · e24992f · e24992f
1 parent fbd5227
commit e24992f
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 2 deletions.
diff --git a/embedding/embedlite/embedshared/EmbedLitePuppetWidget.cpp b/embedding/embedlite/embedshared/EmbedLitePuppetWidget.cpp
@@ -94,7 +94,7 @@ EmbedLitePuppetWidget::EmbedLitePuppetWidget(EmbedLiteWindowBaseChild* window,
   , mDPI(-1.0)
 {
   MOZ_COUNT_CTOR(EmbedLitePuppetWidget);
-  LOGT("this:%p", this);
+  LOGT("Puppet: %p, view: %p, window: %p", this, mView, mWindow);
   InitPrefs();
 }
 
@@ -198,7 +198,8 @@ EmbedLitePuppetWidget::Create(nsIWidget*        aParent,
                               const LayoutDeviceIntRect& aRect,
                               nsWidgetInitData* aInitData)
 {
-  LOGT();
+  LOGT("Puppet: %p, parent: %p", this, aParent);
+
   NS_ASSERTION(!aNativeParent, "got a non-Puppet native parent");
 
   mParent = static_cast<EmbedLitePuppetWidget*>(aParent);
@@ -429,6 +430,8 @@ EmbedLitePuppetWidget::GetNativeData(uint32_t aDataType)
 NS_IMETHODIMP
 EmbedLitePuppetWidget::DispatchEvent(WidgetGUIEvent* event, nsEventStatus& aStatus)
 {
+  LOGT();
+
   aStatus = nsEventStatus_eIgnore;
 
   nsIWidgetListener* listener =

diff --git a/embedding/embedlite/embedshared/EmbedLiteViewBaseChild.cpp b/embedding/embedlite/embedshared/EmbedLiteViewBaseChild.cpp
@@ -177,6 +177,7 @@ EmbedLiteViewBaseChild::InitGeckoWindow(const uint32_t& parentId, const bool& is
   }
 
   mWidget = new EmbedLitePuppetWidget(this);
+  LOGT("puppet widget: %p", static_cast<EmbedLitePuppetWidget*>(mWidget.get()));
   nsWidgetInitData  widgetInit;
   widgetInit.clipChildren = true;
   widgetInit.clipSiblings = true;
@@ -347,6 +348,7 @@ EmbedLiteViewBaseChild::GetInputContext(int32_t* IMEEnabled,
 
 void EmbedLiteViewBaseChild::ResetInputState()
 {
+  LOGT();
   if (!mIMEComposing) {
     return;
   }
@@ -377,6 +379,7 @@ EmbedLiteViewBaseChild::UpdateZoomConstraints(const uint32_t& aPresShellId,
                                               const ViewID& aViewId,
                                               const Maybe<ZoomConstraints> &aConstraints)
 {
+  LOGT();
   return SendUpdateZoomConstraints(aPresShellId,
                                    aViewId,
                                    aConstraints);
@@ -385,6 +388,7 @@ EmbedLiteViewBaseChild::UpdateZoomConstraints(const uint32_t& aPresShellId,
 void
 EmbedLiteViewBaseChild::RelayFrameMetrics(const FrameMetrics& aFrameMetrics)
 {
+  LOGT();
   for (unsigned int i = 0; i < mControllerListeners.Length(); i++) {
     mControllerListeners[i]->RequestContentRepaint(aFrameMetrics);
   }
@@ -768,6 +772,7 @@ EmbedLiteViewBaseChild::RecvHandleScrollEvent(const bool &isRootScrollFrame,
 bool
 EmbedLiteViewBaseChild::RecvUpdateFrame(const FrameMetrics& aFrameMetrics)
 {
+  LOGT();
   if (!mWebBrowser) {
     return true;
   }

diff --git a/embedding/embedlite/embedshared/EmbedLiteViewBaseParent.cpp b/embedding/embedlite/embedshared/EmbedLiteViewBaseParent.cpp
@@ -260,6 +260,7 @@ EmbedLiteViewBaseParent::RecvUpdateZoomConstraints(const uint32_t& aPresShellId,
                                                      const ViewID& aViewId,
                                                      const Maybe<ZoomConstraints>& aConstraints)
 {
+  LOGT("manager: %p layer id: %d", mController->GetManager(), mRootLayerTreeId);
   if (mController->GetManager()) {
     mController->GetManager()->UpdateZoomConstraints(ScrollableLayerGuid(mRootLayerTreeId, aPresShellId, aViewId),
                                                     aConstraints);

diff --git a/embedding/embedlite/utils/TabChildHelper.cpp b/embedding/embedlite/utils/TabChildHelper.cpp
@@ -443,6 +443,7 @@ TabChildHelper::DoUpdateZoomConstraints(const uint32_t& aPresShellId,
                                         const ViewID& aViewId,
                                         const Maybe<mozilla::layers::ZoomConstraints> &aConstraints)
 {
+  LOGT();
   return mView->UpdateZoomConstraints(aPresShellId,
                                       aViewId,
                                       aConstraints);