Print activity state when Ctrl+Shift+Alt+W is pressed

Print activity state when Ctrl+Shift+Alt+W is pressed

This should be useful when debugging resource management.

BUG=None
TBR=sky@chromium.org as an owner of new DEPS dependency ui/wm/public

Committed: https://crrev.com/999736ca77e02f4cb62b4a0c6b1739ffde7c2988
Cr-Commit-Position: refs/heads/master@{#302029}

[hashimoto, 2014-10-28 10:21:26]

hashimoto@chromium.org changed reviewers:
+ oshima@chromium.org

[hashimoto, 2014-10-28 10:21:27]

The output looks like:

[25313:25313:1028/191921:ERROR:screen_accelerator_handler.cc(81)] RootWindow :
RootWindow (0x2571c32c5620) type=0  visible 0,0 1280x720
   AthenaBackground (0x2571c32c5ba0) type=0  visible 0,0 1280x720
      BackgroundWidget (0x2571c32c4020) type=2  visible 0,0 1280x720
   DefaultContainer (0x2571c32c58e0) type=0  visible 0,0 1280x720
      WebActivity (0x2571c5e4c020) type=1  visible 0,0 1280x714 <activity:
state=1 visible=0 media_state=0>
         NativeViewHostAuraClip (0x2571c5a6b648) type=3  0,25 1280x689
            WebContentsViewAura (0x2571c5e4cde0) type=3  0,0 1280x689
               RenderWidgetHostViewAura (0x2571c5e4d0a0) type=3  0,0 1280x689
      WebActivity (0x2571c60ac0a0) type=1  visible 0,0 1280x714 <activity:
state=0 visible=1 media_state=0>
         NativeViewHostAuraClip (0x2571c5546848) type=3  visible 5,55 1270x654
            WebContentsViewAura (0x2571c60ab2e0) type=3  visible 0,0 1280x689
               RenderWidgetHostViewAura (0x2571c60ab5a0) type=3  visible 0,0
1280x689
      WebActivity (0x2571c60d0360) type=1  visible 0,0 1280x714 <activity:
state=0 visible=1 media_state=0>
         NativeViewHostAuraClip (0x2571c5544548) type=3  visible 5,55 1270x654
            WebContentsViewAura (0x2571c60cf5a0) type=3  visible 0,0 1280x689
               RenderWidgetHostViewAura (0x2571c60cf860) type=3  visible 0,0
1280x689
      WebActivity (0x2571c65b0de0) type=1  visible 0,0 1280x714 <activity:
state=0 visible=1 media_state=0>
         NativeViewHostAuraClip (0x2571c5543f48) type=3  visible 5,55 1270x654
            WebContentsViewAura (0x2571c65b0860) type=3  visible 0,0 1280x689
               RenderWidgetHostViewAura (0x2571c65b0b20) type=3  visible 0,0
1280x689
   HomeCardContainer (0x2571c3506ba0) type=0  visible 0,0 1280x720
      "" (0x2571c5e4c860) type=2 [active]  visible 0,620 1280x720
   VirtualKeyboardContainer (0x2571c5850b20) type=0  visible 0,0 1280x720
      KeyboardContainer (0x2571c5850860) type=0  visible 0,0 1280x720

[oshima, 2014-10-28 16:39:03]

https://codereview.chromium.org/687533002/diff/1/athena/screen/screen_acceler...
File athena/screen/screen_accelerator_handler.cc (right):

https://codereview.chromium.org/687533002/diff/1/athena/screen/screen_acceler...
athena/screen/screen_accelerator_handler.cc:8: #include
"athena/activity/public/activity_manager.h"
ugh, screen shouldn't depend no activity.
adding delegate just this would be overkill (although I'm ok with it if you want
that way).

Maybe we should move the debug command to athena/main?
I'm ok to assume ScreenManager::GetContext() returns root window there.

[hashimoto, 2014-10-29 04:54:26]

Patchset #2 (id:20001) has been deleted

[hashimoto, 2014-10-29 04:57:07]

Patchset #2 (id:40001) has been deleted

[hashimoto, 2014-10-29 04:57:36]

https://codereview.chromium.org/687533002/diff/1/athena/screen/screen_acceler...
File athena/screen/screen_accelerator_handler.cc (right):

https://codereview.chromium.org/687533002/diff/1/athena/screen/screen_acceler...
athena/screen/screen_accelerator_handler.cc:8: #include
"athena/activity/public/activity_manager.h"
On 2014/10/28 16:39:02, oshima wrote:
> ugh, screen shouldn't depend no activity.
> adding delegate just this would be overkill (although I'm ok with it if you
want
> that way).
> 
> Maybe we should move the debug command to athena/main?
> I'm ok to assume ScreenManager::GetContext() returns root window there.

Added a new class DebugAcceleratorHandler under athena/main

[oshima, 2014-10-29 14:57:57]

lgtm

[hashimoto, 2014-10-30 03:27:58]

The CQ bit was checked by hashimoto@chromium.org

[commit-bot: I haz the power, 2014-10-30 03:28:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/687533002/60001

[commit-bot: I haz the power, 2014-10-30 03:32:25]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-30 03:32:26]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[hashimoto, 2014-10-30 03:35:48]

hashimoto@chromium.org changed reviewers:
+ sky@chromium.org

[hashimoto, 2014-10-30 03:36:02]

TBRing sky@chromium.org as an owner of new DEPS dependency ui/wm/public

[hashimoto, 2014-10-30 03:36:05]

The CQ bit was checked by hashimoto@chromium.org

[commit-bot: I haz the power, 2014-10-30 03:36:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/687533002/60001

[commit-bot: I haz the power, 2014-10-30 04:09:47]

Committed patchset #2 (id:60001)

[commit-bot: I haz the power, 2014-10-30 04:10:35]

Patchset 2 (id:??) landed as
https://crrev.com/999736ca77e02f4cb62b4a0c6b1739ffde7c2988
Cr-Commit-Position: refs/heads/master@{#302029}