Brian Grinstead [:bgrins] Reporter
	Description • 6 years ago

STR:
* ./mach run --setpref devtools.browserconsole.html=true --jsconsole
* Hover the clear messages (trash can) icon in the Browser Console

Expected:
Tooltip text shows up

Actual:
No tooltip text shows up

The hover text does show up when running inside the normal toolbox, I'm assuming because of `this.frame.tooltip = "aHTMLTooltip";` in toolbox-hosts.js.

	Mats Palmgren (inactive)
	Comment 2 • 6 years ago
mozreview-review

Comment on attachment 8989558 [details]
Bug 1460691 - Support tooltips in top level non-XUL windows. +6102

https://reviewboard.mozilla.org/r/254586/#review261472

r+ on the CanvasFrame changes, but please get enndeakin or someone else to review the rest (I'm going on vacation so I don't have time to page that code in)

::: layout/generic/nsCanvasFrame.cpp:341
(Diff revision 1)
>  void
>  nsCanvasFrame::SetDefaultTooltip(Element* aTooltip)
>  {
> -  NS_WARNING("SetDefaultTooltip not implemented");
> -  return;
> +  MOZ_ASSERT(!aTooltip || aTooltip == mTooltipContent,
> +             "Default tooltip should be anonymous content tooltip.");
> +  aTooltip = mTooltipContent;

This assignment should be the other way around.

	Brendan Dahl [:bdahl] Assignee
	Comment 3 • 6 years ago

Comment on attachment 8989558 [details]
Bug 1460691 - Support tooltips in top level non-XUL windows. +6102

See above^

	Neil Deakin
	Comment 5 • 6 years ago
mozreview-review

Comment on attachment 8989558 [details]
Bug 1460691 - Support tooltips in top level non-XUL windows. +6102

https://reviewboard.mozilla.org/r/254586/#review263514

::: layout/xul/nsXULTooltipListener.cpp:627
(Diff revision 2)
>        }
>      }
>    }
>  
>  #ifdef MOZ_XUL
> -  // titletips should just use the default tooltip
> +  // titletips and non-XUL documents should just use the default tooltip

The tooltip listener doesn't have any specific checks for XUL elements (most specifically in GetTooltipFor), so if the various attributes it retrieves are not on XUL elements, it would check attributes that shouldn't have any meaning.

Is that desired?

	Brendan Dahl [:bdahl] Assignee
	Comment 7 • 6 years ago

Neil,
Just want to make sure you're alright with the new changes^

	Pulsebot
	Comment 8 • 6 years ago

Pushed by bdahl@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b2d89deed396
Support tooltips in top level non-XUL windows. r=enndeakin+6102,mats+6102

	Brendan Dahl [:bdahl] Assignee
	Comment 9 • 6 years ago

I'm going to out of the office the next few weeks, so I went ahead and landed the patch to unblock Brian. Brian mentioned he's fine to do any follow up if there's anything while I'm away.

	Natalia Csoregi [:nataliaCs]
	Comment 10 • 6 years ago
bugherder

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