DevTools: remove unused UI related code

DevTools: remove unused UI related code

BUG=none

Review-Url: https://codereview.chromium.org/2895733002
Cr-Commit-Position: refs/heads/master@{#473679}
Committed: https://chromium.googlesource.com/chromium/src/+/0c4988c82aff00b563b13d66fe10a5e8df9236f4

[luoe, 2017-05-22 17:50:47]

luoe@chromium.org changed reviewers:
+ alph@chromium.org

[luoe, 2017-05-22 17:50:48]

Please take a look

https://codereview.chromium.org/2895733002/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/ui/ContextMenu.js (left):

https://codereview.chromium.org/2895733002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/ui/ContextMenu.js:391:
UI._contextMenu = this;
Added:
https://chromium.googlesource.com/chromium/src/+/5189f94c96828d4422412f19411f...
Used to have a reference to the last shown ContextMenu so that
contextmenuitemselected calls from the Host
would route to the correct context menu.

Now, contextmenus listen for Host events themselves.

https://codereview.chromium.org/2895733002/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/ui/FilterBar.js (left):

https://codereview.chromium.org/2895733002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/ui/FilterBar.js:144: };
Added: https://codereview.chromium.org/33143002
Removed: https://codereview.chromium.org/2482593003

https://codereview.chromium.org/2895733002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/ui/FilterBar.js:455:
UI.ComboBoxFilterUI = class extends Common.Object {
Added: https://codereview.chromium.org/33143002
Removed: https://codereview.chromium.org/2686993002

https://codereview.chromium.org/2895733002/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/ui/KeyboardShortcut.js (left):

https://codereview.chromium.org/2895733002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/ui/KeyboardShortcut.js:292:
UI.KeyboardShortcut.SelectAll = UI.KeyboardShortcut.makeKey('a',
UI.KeyboardShortcut.Modifiers.CtrlOrMeta);
Added, no longer used:
https://chromium.googlesource.com/chromium/src/+/322d83751ad3afee1558bf28349c...

https://codereview.chromium.org/2895733002/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/ui/Panel.js (left):

https://codereview.chromium.org/2895733002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/ui/Panel.js:110:
UI.Panel.counterRightMargin = 25;
Added: 
https://chromium.googlesource.com/chromium/src/+/c0305576fdbe74e069ebc26bbdeb...
When Timeline had a records counter in the drawer, this margin was used in
Drawer.js to animate a slide to the right.

[alph, 2017-05-22 18:53:08]

The CQ bit was checked by alph@chromium.org

[alph, 2017-05-22 18:53:09]

lgtm

[commit-bot: I haz the power, 2017-05-22 18:53:38]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[luoe, 2017-05-22 18:53:38]

Thanks for the quick review!

[commit-bot: I haz the power, 2017-05-22 20:38:10]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495479188585060, "parent_rev":
"902047d6169a2f5cd685b8d13d2c06bc22839e74", "commit_rev":
"0c4988c82aff00b563b13d66fe10a5e8df9236f4"}

[commit-bot: I haz the power, 2017-05-22 20:38:20]

Description was changed from

==========
DevTools: remove unused UI related code

BUG=none
==========

to

==========
DevTools: remove unused UI related code

BUG=none

Review-Url: https://codereview.chromium.org/2895733002
Cr-Commit-Position: refs/heads/master@{#473679}
Committed:
https://chromium.googlesource.com/chromium/src/+/0c4988c82aff00b563b13d66fe10...
==========

[commit-bot: I haz the power, 2017-05-22 20:38:23]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/0c4988c82aff00b563b13d66fe10...