Bug 97559 - MediaStream API: Enhance MediaConstraints to make it easier to get the constraint data
Summary: MediaStream API: Enhance MediaConstraints to make it easier to get the constr...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Tommy Widenflycht
URL:
Keywords:
Depends on:
Blocks: 56459
  Show dependency treegraph
 
Reported: 2012-09-25 05:25 PDT by Tommy Widenflycht
Modified: 2012-09-27 08:01 PDT (History)
9 users (show)

See Also:


Attachments
Patch (13.85 KB, patch)
2012-09-25 05:36 PDT, Tommy Widenflycht
abarth: review+
Details | Formatted Diff | Diff
Patch that belongs to another bug. (41.26 KB, text/plain)
2012-09-26 01:11 PDT, Tommy Widenflycht
no flags Details
Patch for landing (13.90 KB, patch)
2012-09-27 04:51 PDT, Tommy Widenflycht
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tommy Widenflycht 2012-09-25 05:25:09 PDT
Instead of just returning the names, return a pair of name and value.
Comment 1 Tommy Widenflycht 2012-09-25 05:36:29 PDT
Created attachment 165590 [details]
Patch
Comment 2 WebKit Review Bot 2012-09-25 05:38:26 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.
Comment 3 Peter Beverloo (cr-android ews) 2012-09-25 05:52:35 PDT
Comment on attachment 165590 [details]
Patch

Attachment 165590 [details] did not pass cr-android-ews (chromium-android):
Output: http://queues.webkit.org/results/14033036
Comment 4 Tommy Widenflycht 2012-09-25 05:57:03 PDT
Hmm, can really see how my patch could have broken cr-android :/

out/Release/obj.target/Source/WebCore/WebCore.gyp/libwebcore_remaining.a(InspectorDebuggerAgent.o): in function .LTHUNK29:Source/WebCore/inspector/InspectorDebuggerAgent.cpp:633: error: undefined reference to 'WebCore::InspectorFrontend::Debugger::scriptParsed(WTF::String const&, WTF::String const&, int, int, int, int, bool const*, WTF::String const*)'
collect2: ld returned 1 exit status
Comment 5 Tommy Widenflycht 2012-09-25 06:07:25 PDT
s/can/can't/
Comment 6 WebKit Review Bot 2012-09-25 09:32:49 PDT
Comment on attachment 165590 [details]
Patch

Rejecting attachment 165590 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2

Last 500 characters of output:
WebCore/platform/mediastream/MediaConstraints.h
patching file Tools/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file Tools/DumpRenderTree/chromium/MockWebRTCPeerConnectionHandler.cpp
Hunk #1 FAILED at 120.
1 out of 1 hunk FAILED -- saving rejects to file Tools/DumpRenderTree/chromium/MockWebRTCPeerConnectionHandler.cpp.rej

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--force', u'--reviewer', u'Adam Barth']" exit_code: 1 cwd: /mnt/git/webkit-commit-queue

Full output: http://queues.webkit.org/results/14002812
Comment 7 Tommy Widenflycht 2012-09-26 01:11:12 PDT
Created attachment 165745 [details]
Patch that belongs to another bug.
Comment 8 Tommy Widenflycht 2012-09-27 04:51:08 PDT
Created attachment 165977 [details]
Patch for landing
Comment 9 WebKit Review Bot 2012-09-27 07:56:38 PDT
Comment on attachment 165977 [details]
Patch for landing

Clearing flags on attachment: 165977

Committed r129764: <http://trac.webkit.org/changeset/129764>