[Chromium]Fix popup menu RTL bug introduced by Changeset 75982.
Created attachment 80873 [details] Patch
Comment on attachment 80873 [details] Patch Could you review?
Attachment 80873 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1 Source/WebCore/ChangeLog:8: Line contains tab character. [whitespace/tab] [5] Source/WebCore/ChangeLog:10: Line contains tab character. [whitespace/tab] [5] Total errors found: 2 in 3 files If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 80873 [details] Patch Please fix style errors. The code looks good.
Created attachment 80899 [details] Delete tabs My vim automatically inserted tabs. I'll check my environment...
Created attachment 80900 [details] Delete tabs
Comment on attachment 80900 [details] Delete tabs ok
Comment on attachment 80900 [details] Delete tabs Clearing flags on attachment: 80900 Committed r77370: <http://trac.webkit.org/changeset/77370>
All reviewed patches have been landed. Closing bug.
Created attachment 82111 [details] Patch
Comment on attachment 82111 [details] Patch Tamaru-san, Sorry for bothering you. Could you review again? As I wrote in ChangeLog, this is not good for the chrome bug http://codereview.chromium.org/6024008/ Thanks,
Please don't make a committed patch obsolete. Please file another bug.
Ok, (In reply to comment #12) > Please don't make a committed patch obsolete. > Please file another bug.