Bug 122096

Summary: [EFL] Gardening after the resolution of bug 84835
Product: WebKit Reporter: Rob Płóciennik <r.plociennik>
Component: WebKit EFLAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, commit-queue, g.czajkowski, gyuyoung.kim, lucas.de.marchi, rakuco
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Proposed patch
none
Proposed patch
commit-queue: commit-queue-
Yet another attempt at submitting the proposed patch. none

Description Rob Płóciennik 2013-09-30 02:58:24 PDT
Removed expected failures for bug 84835.
editing/pasteboard/copy-inside-h1-preserves-h1 and editing/pasteboard/copy-without-selection are still failing due to bug 86623 and have been marked accordingly.
Comment 1 Rob Płóciennik 2013-09-30 03:06:57 PDT
Created attachment 212970 [details]
Proposed patch
Comment 2 WebKit Commit Bot 2013-09-30 03:08:16 PDT
Attachment 212970 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/platform/efl-wk1/TestExpectations', u'LayoutTests/platform/efl/TestExpectations']" exit_code: 1
LayoutTests/ChangeLog:1:  ChangeLog entry has no bug number  [changelog/bugnumber] [5]
Total errors found: 1 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Rob Płóciennik 2013-09-30 03:18:33 PDT
Created attachment 212973 [details]
Proposed patch
Comment 4 WebKit Commit Bot 2013-09-30 04:01:15 PDT
Comment on attachment 212973 [details]
Proposed patch

Rejecting attachment 212973 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.appspot.com', '--bot-id=webkit-cq-03', 'validate-changelog', '--check-oops', '--non-interactive', 212973, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

ChangeLog entry in LayoutTests/ChangeLog contains OOPS!.

Full output: http://webkit-queues.appspot.com/results/2640167
Comment 5 Rob Płóciennik 2013-09-30 04:11:04 PDT
Created attachment 212975 [details]
Yet another attempt at submitting the proposed patch.
Comment 6 Grzegorz Czajkowski 2013-09-30 04:17:51 PDT
Comment on attachment 212975 [details]
Yet another attempt at submitting the proposed patch.

Thanks for the changes!
Comment 7 WebKit Commit Bot 2013-09-30 04:48:03 PDT
Comment on attachment 212975 [details]
Yet another attempt at submitting the proposed patch.

Clearing flags on attachment: 212975

Committed r156646: <http://trac.webkit.org/changeset/156646>
Comment 8 WebKit Commit Bot 2013-09-30 04:48:06 PDT
All reviewed patches have been landed.  Closing bug.