[Webkit-unassigned] [Bug 45375] [Regression][Gtk] Left and Right Arrows no longer function when caret browsing is enabled

Thu Sep 23 01:38:44 PDT 2010

https://bugs.webkit.org/show_bug.cgi?id=45375

Mario Sanchez Prada <msanchez at igalia.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |msanchez at igalia.com

--- Comment #10 from Mario Sanchez Prada <msanchez at igalia.com>  2010-09-23 01:38:44 PST ---
(In reply to comment #9)
> (From update of attachment 68353 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=68353&action=review
> 
> > LayoutTests/platform/gtk/editing/selection/caret-mode-paragraph-keys-navigation.html:5
> > +<script>
> 
> this pointer to js-test-style requires one ".."

I don't get this. It's precisely like that in the current patch:

   [...]
   <head>
   <link rel="stylesheet" href="../fast/js/resources/js-test-style.css">
   <script>
   [...]

> the other two references require
> 
> "../../../../"

Yep, and it's like that:

   [...]
   </script>
   <script src="../../../../fast/js/resources/js-test-pre.js"></script>
   </head>
   [...]
   </script>
   <script src="../../../../fast/js/resources/js-test-post.js"></script>
   </body>
   </html>

It was "../../.." when this test was inside platform/gtk/accessibility, but now it is under platform/gtk/editing/selection it's obvious we need an extra "../" :-)

> not sure if the js-test-pre stuff is getting loaded
>
> > LayoutTests/platform/gtk/editing/selection/caret-mode-paragraph-keys-navigation.html:8
> > +<script src="../../../../fast/js/resources/js-test-pre.js"></script>
> 
> as in here

Yes, it is being loaded. If it wasn't the description() and shouldBe() functions would be missing and whole text wouldn't work

> > LayoutTests/platform/gtk/editing/selection/caret-mode-paragraph-keys-navigation.html:70
> > +<script src="../../../../fast/js/resources/js-test-post.js"></script>
> 
> and here

Same for this.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.