Bug 78751 – Add support for the translate attribute in html elements.

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[Pablo Flouret]

http://www.whatwg.org/specs/web-apps/current-work/multipage/elements.html#attr-translate

[Pablo Flouret]

Created attachment 127249 [details]
Patch

[Adam Barth]

Comment on attachment 127249 [details]
Patch

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

Is there any more to this attribute?  A link to the spec would be helpful so I can read up on it easily.  Some nits below.

> Source/WebCore/ChangeLog:8
> +        Add support for the translate attribute in html elements.
> +        https://bugs.webkit.org/show_bug.cgi?id=78751
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +        Test: fast/dom/HTMLElement/translate.html

Can you add some more information to the ChangeLog, like a link to the spec?

> Source/WebCore/html/HTMLElement.cpp:823
> +            if (mode != TranslateAttributeInherit)
> +                return mode == TranslateAttributeYes;

Should we ASSERT that the mode is either TranslateAttributeYes or TranslateAttributeNo in the body of this if?

[Pablo Flouret]

(In reply to comment #2)
> (From update of attachment 127249 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=127249&action=review
> 
> Is there any more to this attribute?  A link to the spec would be helpful so I can read up on it easily.  Some nits below.

The link to the spec is in the description field of this bug, to wit, http://www.whatwg.org/specs/web-apps/current-work/multipage/elements.html#attr-translate

I forgot to add it to the changelog, will do.

> 
> > Source/WebCore/html/HTMLElement.cpp:823
> > +            if (mode != TranslateAttributeInherit)
> > +                return mode == TranslateAttributeYes;
> 
> Should we ASSERT that the mode is either TranslateAttributeYes or TranslateAttributeNo in the body of this if?

We could.

[Pablo Flouret]

Created attachment 127260 [details]
Fixed changelogs and added assert

[Adam Barth]

Comment on attachment 127260 [details]
Fixed changelogs and added assert

Thanks!

[WebKit Review Bot]

Comment on attachment 127260 [details]
Fixed changelogs and added assert

Rejecting attachment 127260 [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:
ools/ChangeLog
CONFLICT (content): Merge conflict in Tools/ChangeLog
Failed to merge in the changes.
Patch failed at 0001 [chromium] Rebaseline JPEG image results after r107389

When you have resolved this problem run "git rebase --continue".
If you would prefer to skip this patch, instead run "git rebase --skip".
To restore the original branch and stop rebasing run "git rebase --abort".

rebase refs/remotes/origin/master: command returned error: 1

Died at Tools/Scripts/update-webkit line 164.

Full output: http://queues.webkit.org/results/11527988

[WebKit Review Bot]

Comment on attachment 127260 [details]
Fixed changelogs and added assert

Clearing flags on attachment: 127260

Committed r107883: <http://trac.webkit.org/changeset/107883>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.

[Simon Fraser (smfr)]

This broke mobile.orange.fr: bug 82246.

[rakesh makluri]

Hi Adam,

  With this support our product is breaking now. Present we are loading and using the translate() API on HTML elements to translate. But now with this change it is breaking. Is there any way to override this attribute.

  We can change the API but, so many existing customers and our internal components are also using this.
  Please help with this.