Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(528)

Side by Side Diff: third_party/WebKit/Source/devtools/front_end/elements/ElementsTreeOutline.js

Issue 2712063002: [DevTools] Migrate Popover to GlassPane (Closed)
Patch Set: nit Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2007, 2008 Apple Inc. All rights reserved. 2 * Copyright (C) 2007, 2008 Apple Inc. All rights reserved.
3 * Copyright (C) 2008 Matt Lilek <webkit@mattlilek.com> 3 * Copyright (C) 2008 Matt Lilek <webkit@mattlilek.com>
4 * Copyright (C) 2009 Joseph Pecoraro 4 * Copyright (C) 2009 Joseph Pecoraro
5 * 5 *
6 * Redistribution and use in source and binary forms, with or without 6 * Redistribution and use in source and binary forms, with or without
7 * modification, are permitted provided that the following conditions 7 * modification, are permitted provided that the following conditions
8 * are met: 8 * are met:
9 * 9 *
10 * 1. Redistributions of source code must retain the above copyright 10 * 1. Redistributions of source code must retain the above copyright
(...skipping 570 matching lines...) Expand 10 before | Expand all | Expand 10 after
581 node, Components.DOMPresentationUtils.buildImagePreviewContents.bind( 581 node, Components.DOMPresentationUtils.buildImagePreviewContents.bind(
582 Components.DOMPresentationUtils, node.target(), link[Elements. ElementsTreeElement.HrefSymbol], true, 582 Components.DOMPresentationUtils, node.target(), link[Elements. ElementsTreeElement.HrefSymbol], true,
583 showPopover)); 583 showPopover));
584 584
585 /** 585 /**
586 * @param {!Element=} contents 586 * @param {!Element=} contents
587 */ 587 */
588 function showPopover(contents) { 588 function showPopover(contents) {
589 if (!contents) 589 if (!contents)
590 return; 590 return;
591 popover.setCanShrink(false);
592 popover.showForAnchor(contents, link); 591 popover.showForAnchor(contents, link);
593 } 592 }
594 } 593 }
595 594
596 _onmousedown(event) { 595 _onmousedown(event) {
597 var element = this._treeElementFromEvent(event); 596 var element = this._treeElementFromEvent(event);
598 597
599 if (!element || element.isEventWithinDisclosureTriangle(event)) 598 if (!element || element.isEventWithinDisclosureTriangle(event))
600 return; 599 return;
601 600
(...skipping 1087 matching lines...) Expand 10 before | Expand all | Expand 10 after
1689 */ 1688 */
1690 function resolved(node) { 1689 function resolved(node) {
1691 if (node) { 1690 if (node) {
1692 this.treeOutline._selectedDOMNode = node; 1691 this.treeOutline._selectedDOMNode = node;
1693 this.treeOutline._selectedNodeChanged(); 1692 this.treeOutline._selectedNodeChanged();
1694 } 1693 }
1695 } 1694 }
1696 return true; 1695 return true;
1697 } 1696 }
1698 }; 1697 };
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698