940155 - australis menu support for social marks button

Status: VERIFIED FIXED

(Firefox Graveyard :: SocialAPI, defect)

Description

[Shane Caraveo (:mixedpuppy)]

split out from bug 914921 to simplify the process.  Need to make the social mark button work in australis

Comment 1

[Shane Caraveo (:mixedpuppy)]

Attachment: make marks button work in menu panel

https://tbpl.mozilla.org/?tree=Try&rev=94c8bacd998f

This basically follows how the status button changes worked in bug 914921, though a listener would not work in this case, something to do with having a custom button and the widget wrapping on it.  Watching for the menu panel being shown is good enough as an update mechanism for this button.

Comment 2

[Mark Hammond [:markh] [:mhammond]]

Comment on attachment 8340173 [details] [diff] [review]
make marks button work in menu panel

Review of attachment 8340173 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/base/content/socialmarks.xml
@@ +150,3 @@
>            // add our resizer after the dom is ready
> +          if (!this.inMenuPanel) {
> +            let DynamicResizeWatcher = Cu.import("resource:///modules/Social.jsm", {}).DynamicResizeWatcher;

I know it wasn't before, but I think a lazy getter at the top of the file would be better.

@@ +251,5 @@
>          // pageData is passed into us. In this case, we always load the iframe
>          // and show it since the url may not be the browser tab, but an image,
>          // link, etc. inside the page. We also "update" the iframe to the
>          // previous url when it is closed.
> +        let panel = this.panel;

this is unused

@@ +304,5 @@
> +            if (!link || !link.ownerDocument || !rel || !link.href)
> +              return;
> +            if (link.rel.indexOf("icon") < 0)
> +              return;
> +    

trailing whitespace here and a few lines down

Comment 3

[Shane Caraveo (:mixedpuppy)]

Attachment: make marks button work in menu panel

updated with comments, new try https://tbpl.mozilla.org/?tree=Try&rev=d0b206ab50d3

https://hg.mozilla.org/integration/fx-team/rev/babe33b649a3

Comment 4

[Shane Caraveo (:mixedpuppy)]

> updated with comments, new try
> https://tbpl.mozilla.org/?tree=Try&rev=d0b206ab50d3

mixing bugs/patches, try is for something else, previous try was in comment 1 is for this patch

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/babe33b649a3

Comment 6

[Cornel Ionce [:noni] [Hubs QA]]

Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:29.0) Gecko/20100101 Firefox/29.0

Verified fixed on latest Aurora (build ID: 20140310004003).