Improve rcube_find_object() performance by prefering getElementById() as th...

Aleksander Machniak

2014-05-02 c0037f1882dd3b669e0ac49dac272cf26435015e

 program/js/common.js

@@ -224,7 +224,7 @@
    p.element = document;

  if (!p.object._rc_events)
    p.object._rc_events = [];
    p.object._rc_events = {};

  var key = p.event + '*' + p.method;
  if (!p.object._rc_events[key])
@@ -472,21 +472,25 @@
function rcube_find_object(id, d)
{
  var n, f, obj, e;
  if(!d) d = document;

  if(d.getElementsByName && (e = d.getElementsByName(id)))
  if (!d) d = document;

  if (d.getElementById)
    if (obj = d.getElementById(id))
      return obj;

  if (!obj && d.getElementsByName && (e = d.getElementsByName(id)))
    obj = e[0];
  if(!obj && d.getElementById)
    obj = d.getElementById(id);
  if(!obj && d.all)

  if (!obj && d.all)
    obj = d.all[id];

  if(!obj && d.images.length)
  if (!obj && d.images.length)
    obj = d.images[id];

  if (!obj && d.forms.length) {
    for (f=0; f<d.forms.length; f++) {
      if(d.forms[f].name == id)
      if (d.forms[f].name == id)
        obj = d.forms[f];
      else if(d.forms[f].elements[id])
        obj = d.forms[f].elements[id];
@@ -494,7 +498,8 @@
  }

  if (!obj && d.layers) {
    if (d.layers[id]) obj = d.layers[id];
    if (d.layers[id])
      obj = d.layers[id];
    for (n=0; !obj && n<d.layers.length; n++)
      obj = rcube_find_object(id, d.layers[n].document);
  }
@@ -508,8 +513,8 @@
  var mouse = rcube_event.get_mouse_pos(ev),
    pos = $(obj).offset();

  return ((mouse.x >= pos.left) && (mouse.x < (pos.left + obj.offsetWidth)) &&
    (mouse.y >= pos.top) && (mouse.y < (pos.top + obj.offsetHeight)));
  return (mouse.x >= pos.left) && (mouse.x < (pos.left + obj.offsetWidth)) &&
    (mouse.y >= pos.top) && (mouse.y < (pos.top + obj.offsetHeight));
};


@@ -521,6 +526,7 @@
      (path ? "; path=" + path : "") +
      (domain ? "; domain=" + domain : "") +
      (secure ? "; secure" : "");

  document.cookie = curCookie;
};