WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
84328
Media Stream API: Piece of code cleanup for the chromium port
https://bugs.webkit.org/show_bug.cgi?id=84328
Summary
Media Stream API: Piece of code cleanup for the chromium port
Jeremy Mao
Reported
2012-04-19 04:33:58 PDT
Created
attachment 137877
[details]
Patch Remove some dead code, please see the attachment.
Attachments
Patch
(3.34 KB, patch)
2012-04-19 04:33 PDT
,
Jeremy Mao
webkit.review.bot
: commit-queue-
Details
Formatted Diff
Diff
Patch
(7.16 KB, patch)
2012-04-19 05:12 PDT
,
Jeremy Mao
no flags
Details
Formatted Diff
Diff
Patch
(7.16 KB, patch)
2012-04-19 05:16 PDT
,
Jeremy Mao
tkent
: review+
Details
Formatted Diff
Diff
Patch
(7.74 KB, patch)
2012-04-19 18:36 PDT
,
Jeremy Mao
tkent
: review+
webkit.review.bot
: commit-queue-
Details
Formatted Diff
Diff
Patch
(8.17 KB, patch)
2012-04-19 19:18 PDT
,
Jeremy Mao
no flags
Details
Formatted Diff
Diff
Patch
(8.17 KB, patch)
2012-04-19 19:48 PDT
,
Jeremy Mao
no flags
Details
Formatted Diff
Diff
Show Obsolete
(5)
View All
Add attachment
proposed patch, testcase, etc.
WebKit Review Bot
Comment 1
2012-04-19 04:35:56 PDT
Please wait for approval from
abarth@webkit.org
,
dglazkov@chromium.org
,
fishd@chromium.org
,
jamesr@chromium.org
or
tkent@chromium.org
before submitting, as this patch contains changes to the Chromium public API. See also
https://trac.webkit.org/wiki/ChromiumWebKitAPI
.
WebKit Review Bot
Comment 2
2012-04-19 04:39:32 PDT
Comment on
attachment 137877
[details]
Patch
Attachment 137877
[details]
did not pass chromium-ews (chromium-xvfb): Output:
http://queues.webkit.org/results/12297282
Jeremy Mao
Comment 3
2012-04-19 05:12:13 PDT
Created
attachment 137880
[details]
Patch
Jeremy Mao
Comment 4
2012-04-19 05:16:17 PDT
Created
attachment 137881
[details]
Patch Fix a typo
Kent Tamura
Comment 5
2012-04-19 16:04:31 PDT
Comment on
attachment 137881
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=137881&action=review
> Tools/DumpRenderTree/chromium/WebUserMediaClientMock.h:53 > + virtual void requestUserMedia(const WebUserMediaRequest&, const WebVector<WebMediaStreamSource>&, const WebVector<WebMediaStreamSource>&);
We had better append 'OVERRIDE' to this.
Jeremy Mao
Comment 6
2012-04-19 18:36:05 PDT
Created
attachment 138023
[details]
Patch
Jeremy Mao
Comment 7
2012-04-19 18:36:57 PDT
(In reply to
comment #5
)
> (From update of
attachment 137881
[details]
) > View in context:
https://bugs.webkit.org/attachment.cgi?id=137881&action=review
> > > Tools/DumpRenderTree/chromium/WebUserMediaClientMock.h:53 > > + virtual void requestUserMedia(const WebUserMediaRequest&, const WebVector<WebMediaStreamSource>&, const WebVector<WebMediaStreamSource>&); > > We had better append 'OVERRIDE' to this.
Fixed
Kent Tamura
Comment 8
2012-04-19 18:39:39 PDT
Comment on
attachment 137881
[details]
Patch please use "webkit-patch upload" to upload a patch.
WebKit Review Bot
Comment 9
2012-04-19 18:40:40 PDT
Comment on
attachment 138023
[details]
Patch Rejecting
attachment 138023
[details]
from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 1 Last 500 characters of output: r_text(self._author_text) File "/mnt/git/webkit-commit-queue/Tools/Scripts/webkitpy/common/checkout/changelog.py", line 156, in _parse_author_text return [ChangeLogEntry._parse_author_name_and_email(author) for author in authors] File "/mnt/git/webkit-commit-queue/Tools/Scripts/webkitpy/common/checkout/changelog.py", line 148, in _parse_author_name_and_email return {'name': match.group("name"), 'email': match.group("email")} AttributeError: 'NoneType' object has no attribute 'group' Full output:
http://queues.webkit.org/results/12460092
Jeremy Mao
Comment 10
2012-04-19 19:18:07 PDT
Created
attachment 138026
[details]
Patch
Kent Tamura
Comment 11
2012-04-19 19:30:43 PDT
Comment on
attachment 138026
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=138026&action=review
> Source/WebKit/chromium/ChangeLog:1 > +2012-04-19 Mao, Yujie <
yujie.mao@intel.com
>
Unfortunately our tool doesn't accept ',' in a person name.
Eric Seidel (no email)
Comment 12
2012-04-19 19:32:40 PDT
Looks like a bug in webkitpy's ChangeLog handling code.
Jeremy Mao
Comment 13
2012-04-19 19:42:04 PDT
(In reply to
comment #11
)
> (From update of
attachment 138026
[details]
) > View in context:
https://bugs.webkit.org/attachment.cgi?id=138026&action=review
> > > Source/WebKit/chromium/ChangeLog:1 > > +2012-04-19 Mao, Yujie <
yujie.mao@intel.com
> > > Unfortunately our tool doesn't accept ',' in a person name.
:) It's ok, I will remove ',' in my name.
Jeremy Mao
Comment 14
2012-04-19 19:48:25 PDT
Created
attachment 138030
[details]
Patch
WebKit Review Bot
Comment 15
2012-04-19 20:29:52 PDT
Comment on
attachment 138030
[details]
Patch Clearing flags on attachment: 138030 Committed
r114714
: <
http://trac.webkit.org/changeset/114714
>
WebKit Review Bot
Comment 16
2012-04-19 20:29:58 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug