WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 139450
[GTK] Minor fixes for documentation and translations
https://bugs.webkit.org/show_bug.cgi?id=139450
Summary
[GTK] Minor fixes for documentation and translations
Tomas Popela
Reported
2014-12-09 10:59:13 PST
Fix a mistype in documentation, mark the zoom-level and is-loading properties name for translation when touching their defition also fix their indentation style.
Attachments
Patch
(3.15 KB, patch)
2014-12-09 11:05 PST
,
Tomas Popela
no flags
Details
Formatted Diff
Diff
Patch
(4.01 KB, patch)
2014-12-10 00:14 PST
,
Tomas Popela
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Tomas Popela
Comment 1
2014-12-09 11:05:22 PST
Created
attachment 242941
[details]
Patch
WebKit Commit Bot
Comment 2
2014-12-09 11:06:18 PST
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See
http://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API
WebKit Commit Bot
Comment 3
2014-12-09 16:32:19 PST
Comment on
attachment 242941
[details]
Patch Rejecting
attachment 242941
[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-01', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 242941, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit Last 500 characters of output: -> origin/master Partial-rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ... Currently at 177042 = a4b0e9806d0f0c4133be110a1a48c4263c7f7231
r177043
= 54e0b7791c859b2019909f9834db550218b06649
r177044
= 5918955cc12fa7ae10897228b3ce6cd85d2dd080 Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc First, rewinding head to replay your work on top of it... Fast-forwarded master to refs/remotes/origin/master. Full output:
http://webkit-queues.appspot.com/results/5438752612679680
Martin Robinson
Comment 4
2014-12-09 16:44:50 PST
Comment on
attachment 242941
[details]
Patch Oops. This patch is missing a ChangeLog. If you use webkit-patch to upload your patch to bugzilla it will take care of this for you.
Tomas Popela
Comment 5
2014-12-10 00:07:32 PST
(In reply to
comment #4
)
> Comment on
attachment 242941
[details]
> Patch > > Oops. This patch is missing a ChangeLog. If you use webkit-patch to upload > your patch to bugzilla it will take care of this for you.
Hmm I used the webkit-patch to upload it.. I will do it again..
Tomas Popela
Comment 6
2014-12-10 00:14:18 PST
Created
attachment 242991
[details]
Patch
Sergio Villar Senin
Comment 7
2014-12-11 02:28:37 PST
Comment on
attachment 242991
[details]
Patch At some point we should address the inconsistencies in the properties definitions inside those files, for example we have different indentations, we use camel case in some property descriptions and not in others...
WebKit Commit Bot
Comment 8
2014-12-11 13:17:11 PST
Comment on
attachment 242991
[details]
Patch Clearing flags on attachment: 242991 Committed
r177177
: <
http://trac.webkit.org/changeset/177177
>
WebKit Commit Bot
Comment 9
2014-12-11 13:17:14 PST
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