Bug 62551

Summary: r87539 broke HTMLMediaElement url logging
Product: WebKit Reporter: Eric Carlson <eric.carlson>
Component: MediaAssignee: Adam Barth <abarth>
Severity: Normal CC: abarth, eric, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
Patch none

Description Eric Carlson 2011-06-13 10:32:52 PDT
HTMLMediaElement's urlForLogging was broken by http://trac.webkit.org/changeset/87539. With logging enabled (defaults write com.apple.Safari WebCoreLogLevel "0x01000000" on OS X), urls are never logged.
Comment 1 Adam Barth 2011-06-13 10:51:57 PDT
What happens instead?
Comment 2 Adam Barth 2011-06-13 10:53:04 PDT
Yeah, that code doesn't may any sense.  It's returning a pointer to a temporary that's out of scope!  Who wrote that!
Comment 3 Adam Barth 2011-06-13 10:54:44 PDT
I blame Eric.  :)

/me will fix.
Comment 4 Eric Carlson 2011-06-13 10:57:13 PDT
(In reply to comment #3)
> I blame Eric.  :)
Hey now, watch it with the finger pointing - you could put out an eye ;-)
Comment 5 Adam Barth 2011-06-13 11:05:03 PDT
Created attachment 96975 [details]
Comment 6 WebKit Review Bot 2011-06-13 11:37:47 PDT
Comment on attachment 96975 [details]

Clearing flags on attachment: 96975

Committed r88662: <http://trac.webkit.org/changeset/88662>
Comment 7 WebKit Review Bot 2011-06-13 11:37:52 PDT
All reviewed patches have been landed.  Closing bug.