Re: Cookie Summary

[Jan Honza Odvarko]

I didn't test is, but try something as follows:

if (!this.table)
return;

var totalSize = 0;

var rows = this.table.getElementsByClassName("cookieRow");
for (var i=0; i<rows.length; i++)
{
var row = rows[i];

var cookie = Firebug.getRepObject(row);
if (cookie)
totalSize += cookie.size;
}


Honza

[Abhiram]

Hello Jan,
                  I have tried the code. Now the cookie panel is not even working. I have commited my changes in the repo. Please checkout.

https://github.com/abhiram92/firebug/blob/master/extension/content/firebug/cookies/cookiePanel.js#L464

The row should return the number of cookies and the size too. I think the problem is with representing it in cookieReps.js
I am struggling with it many hours ago. Yet no result. Thought i could use some help.

Best,
Abhiram

[Jan Honza Odvarko]

On Jul 7, 8:35 am, Abhiram <abhiram.ampabath...@gmail.com> wrote:
> Hello Jan,
>                   I have tried the code. Now the cookie panel is not even
> working. I have commited my changes in the repo. Please checkout.

Well, there are syntax errors in cookieReps.js, on line 131
You need to fix it
Honza

[Jan Honza Odvarko]

On Jul 7, 8:35 am, Abhiram <abhiram.ampabath...@gmail.com> wrote:

> Hello Jan,
>                   I have tried the code. Now the cookie panel is not even
> working. I have commited my changes in the repo. Please checkout.
>

> https://github.com/abhiram92/firebug/blob/master/extension/content/fi...

>
> The row should return the number of cookies and the size too. I think the
> problem is with representing it in cookieReps.js
> I am struggling with it many hours ago. Yet no result. Thought i could use
> some help.

Also note that the code snippet I posted above is not tested.
It's just take from the Net panel see here:

https://github.com/firebug/firebug/blob/master/extension/content/firebug/net/netPanel.js#L1339

Honza