Chat Frame Window Detection

[TULOA]

For my addon it hooks functions for all chat frames.

My problem is when you create a new window from any source the functions arent hooked after load. So I know I need to hook them on the new frames. So I need help with the following:

1) Is there an event that fires upon creating a new chat frame that also passes the frame to hook the stuff that way?
2) How would I grab some sort of unique identifier to know that I am not setting up a second hook for the same frame. I would assume a hook set again will cause it to dup the functions tied to them.

Edit: For Future Reference I am doing the following:

<ChatFrame>.hooked = true

You can test it in wow by running:
/run ChatFrame1.hooked = true
/run print(_G["ChatFrame1"].hooked) --Will display true

[Vrul]

You should be able to just hook ChatFrame_OnLoad since it should only be called once per chat frame:

hooksecurefunc('ChatFrame_OnLoad', function(chatFrame)
    -- do stuff
end)

If you want to be extra sure not to hook the same chat frame twice:

local hooked = { }
hooksecurefunc('ChatFrame_OnLoad', function(chatFrame)
    if hooked[chatFrame] then return end
    hooked[chatFrame] = true
    -- do stuff
end)

[TULOA]

I hadn't thought about that but not long after I came up with a solution.

Adding a custom variable in the chat frame. The reason I chose this way is because if you delete a window and remake it its still counted 1 to NUM_CHAT_WINDOWS.

If you make a window it for example be ChatFrame2. When you delete it and make a new one it will technically be in the list still unless you hook the function if one is called on deletion. When you make another assuming it doesnt delete the addon would assume the frame is already hooked.

With my method when the frame is deleted the variable is gone as well. I was referring mainly on BNet Convos because they dont name themselves like the others. I get 's' for the name. I dont think it applies to addon created ones or others like that but I wasnt gonna take a risk and wanted a universal coding.

[Phanx]

Just process the normal chat frames normally, and hook the function that opens temporary windows. Here's what I do in my chat addon:

	for i = 1, NUM_CHAT_WINDOWS do
		self:ProcessFrame(_G["ChatFrame" .. i])
	end

	hooks.FCF_OpenTemporaryWindow = FCF_OpenTemporaryWindow
	FCF_OpenTemporaryWindow = function(...)
		local frame = hooks.FCF_OpenTemporaryWindow(...)
		self:ProcessFrame(frame)
		return frame
	end

Then my ProcessFrames function looks like this:

local frames = {}

function PhanxChat:ProcessFrame(frame)
	if frames[frame] then return end

	-- do all the stuff here

	frames[frame] = true
end

Since it stores a reference to the frame object itself, there's no need to add keys to the frame that could conceivably be removed, or match names, or anything else.