Bug 117174

Summary: Unable to play videos in Safari using QuickTime plug-in until I've played those videos in Chrome first
Product: WebKit Reporter: Simon Cooper <scooper>
Component: WebKit2Assignee: Simon Cooper <scooper>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, scooper
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
patch
commit-queue: review-, commit-queue: commit-queue-
patch
commit-queue: review-, commit-queue: commit-queue-
patch none

Description Simon Cooper 2013-06-03 18:27:14 PDT
Unable to play videos in Safari using QuickTime plug-in until I've played those videos in Chrome first
Comment 1 Simon Cooper 2013-06-03 18:38:02 PDT
Created attachment 203641 [details]
patch
Comment 2 WebKit Commit Bot 2013-06-03 18:38:32 PDT
Comment on attachment 203641 [details]
patch

Rejecting attachment 203641 [details] from review queue.

scooper@apple.com does not have reviewer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/committers.py.

- If you do not have reviewer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have reviewer rights please correct the error in Tools/Scripts/webkitpy/common/config/committers.py by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your reviewer rights.
Comment 3 WebKit Commit Bot 2013-06-03 18:39:06 PDT
Comment on attachment 203641 [details]
patch

Rejecting attachment 203641 [details] from commit-queue.

scooper@apple.com does not have committer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/committers.py.

- If you do not have committer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have committer rights please correct the error in Tools/Scripts/webkitpy/common/config/committers.py by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your committer rights.
Comment 4 Simon Cooper 2013-06-03 18:40:15 PDT
Created attachment 203642 [details]
patch
Comment 5 WebKit Commit Bot 2013-06-03 18:40:43 PDT
Comment on attachment 203642 [details]
patch

Rejecting attachment 203642 [details] from review queue.

scooper@apple.com does not have reviewer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/committers.py.

- If you do not have reviewer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have reviewer rights please correct the error in Tools/Scripts/webkitpy/common/config/committers.py by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your reviewer rights.
Comment 6 WebKit Commit Bot 2013-06-03 18:41:16 PDT
Comment on attachment 203642 [details]
patch

Rejecting attachment 203642 [details] from commit-queue.

scooper@apple.com does not have committer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/committers.py.

- If you do not have committer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have committer rights please correct the error in Tools/Scripts/webkitpy/common/config/committers.py by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your committer rights.
Comment 7 Simon Cooper 2013-06-03 18:42:21 PDT
Created attachment 203643 [details]
patch
Comment 8 Simon Cooper 2013-06-03 18:43:56 PDT
webkit-upload seems to be adding duplicate ChangeLog entries.  So I’ve manually attached this patch - I’ve used the script previously and manually setting the review and commit-queue flags was “interesting”.
Comment 9 Alexey Proskuryakov 2013-06-03 22:41:35 PDT
Comment on attachment 203643 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=203643&action=review

> Source/WebKit2/Resources/PlugInSandboxProfiles/com.apple.WebKit.plugin-common.sb:161
> +(allow ipc-posix-shm-read* ipc-posix-shm-write-data

Lion effectively gets (allow ipc-posix-shm ipc-posix-shm …), presumably that is fine?
Comment 10 WebKit Commit Bot 2013-06-03 23:03:14 PDT
Comment on attachment 203643 [details]
patch

Clearing flags on attachment: 203643

Committed r151156: <http://trac.webkit.org/changeset/151156>
Comment 11 WebKit Commit Bot 2013-06-03 23:03:18 PDT
All reviewed patches have been landed.  Closing bug.