Changeset 58280 in webkit


Ignore:
Timestamp:
Apr 26, 2010 8:07:38 PM (14 years ago)
Author:
eric@webkit.org
Message:

2010-04-26 Daniel Cheng <​dcheng@chromium.org>

Reviewed by Jian Li.

[chromium] Fix ChromiumDataObject::setURL to not populate its internal URL list with empty URLs.
​https://bugs.webkit.org/show_bug.cgi?id=38159

No new tests.

  • platform/chromium/ChromiumDataObject.h: (WebCore::ChromiumDataObject::setURL):
Location:
trunk/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/WebCore/ChangeLog

    r58276 r58280  
     12010-04-26  Daniel Cheng  <dcheng@chromium.org>
     2
     3        Reviewed by Jian Li.
     4
     5        [chromium] Fix ChromiumDataObject::setURL to not populate its internal URL list with empty URLs.
     6        https://bugs.webkit.org/show_bug.cgi?id=38159
     7
     8        No new tests.
     9
     10        * platform/chromium/ChromiumDataObject.h:
     11        (WebCore::ChromiumDataObject::setURL):
     12
    1132010-04-26  Daniel Cheng  <dcheng@chromium.org>
    214

  • trunk/WebCore/platform/chromium/ChromiumDataObject.h

    r55766 r58280  
    8080            url = newURL;
    8181            uriList.clear();
     82            if (newURL.isEmpty())
     83                return;
    8284            uriList.append(newURL.string());
    8385        }
Note: See TracChangeset for help on using the changeset viewer.