| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
| |
And moves from launcher/ to shelf/ directory.
BUG=248353
TEST=None, no behavior changes.
R=harrym@chromium.org,jamescook@chromium.org
Review URL: https://codereview.chromium.org/45973003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@231738 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also adds a typedef from RootWindow to WindowEventDispatcher, the eventual name of this type.
Adds a GetDispatcher() method that crawls up to the root Window and returns the WindowEventDispatcher.
R=sky@chromium.org
http://crbug.com/308843
Review URL: https://codereview.chromium.org/37733003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@231077 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=303267,276513
TEST=Covered by AppListControllerTest.*
R=flackr@chromium.org, jamescook@chromium.org
Review URL: https://codereview.chromium.org/25708010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@226942 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=276513
TEST=WindowSelectorTest.SelectingHidesAppList
Review URL: https://codereview.chromium.org/25260004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@226322 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
R=ben@chromium.org
TBR=brettw@chromium.org, scherkus@chromium.org, ananta@chromium.org,
Review URL: https://codereview.chromium.org/23851041
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223906 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=288725
TEST=visual
Review URL: https://chromiumcodereview.appspot.com/23440036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223238 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
menu unexpectedly
The problem was as follows:
When the app list menu gets closed it gets closed delayed. The right click menu opens. Once the menu got animated out, it will destruct itself. When doing so it will surrender its activation state to the system and the menu controller will close all menus.
To avoid this I remove the activation state before the app list gets destroyed. (At that point in time it is not active anymore anyways). I also tried to set the focus in the menu runner on its parent - but that widget is not activatable and that does not work.
BUG=285785
TEST=visual
Review URL: https://chromiumcodereview.appspot.com/23457008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@222868 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
applist button.
R=harrym@chromium.org
BUG=287593
TEST=Compiles, Visual test
Review URL: https://chromiumcodereview.appspot.com/23526030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@222215 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
The problem was that the positioning was done before it was shown and the move_with_anchor didn't work since there was no anchor. Changed it now to use an anchor offset.
BUG=284780
TEST=unittest (partial), visually [(RTL and LTR) x (left, right, bottom aligned) x (always visible, sliding into view, hidden)]
Review URL: https://chromiumcodereview.appspot.com/23622020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@222191 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Remove unnecessary includes, forward decls
BUG=272460
Review URL: https://chromiumcodereview.appspot.com/24020002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@221605 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Motivation of this CL:
Only app list shown by clicking app list button enables drag and drop from
app list to launcher because dnd host is only set by app list button click.
Other behaviors such as webapp installing from web store or accelerator show
app list, too. DnD feature also should be enabled with these cases.
R=skuhne@chromium.org
BUG=267410
TEST=Manual test
Review URL: https://chromiumcodereview.appspot.com/21765002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@216265 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
tray menu
A couple of pixels offset.
BUG=264227
TEST=visual
Review URL: https://chromiumcodereview.appspot.com/20487002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@214091 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
application launcher will be flush with the shelf
There is one edge case left for a rotated shelf: Messages are slightly offsetted from the system menu (if existent). Beside that everything seems to work now as expected.
BUG=259029
TEST=visual with L/R languages and with different shelf alignments
Review URL: https://chromiumcodereview.appspot.com/19771021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@213500 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=257000
TBR=sky@chromium.org
Review URL: https://chromiumcodereview.appspot.com/18668002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@211626 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Create a separate flow for the app list icon (behind alternate-shelf-layout flag) whereby the app list icon has different state indication (background images), can be reordered, and defaults to be the first item in the launcher.
BUG=244994
Review URL: https://chromiumcodereview.appspot.com/16831018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@209556 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=245016
TEST=visual
Review URL: https://chromiumcodereview.appspot.com/16909006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@206628 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
RootWindowObserver::OnRootWindowMoved to OnRootWindowHostMoved
- Changed so that this is called only when host window's size
is changed.
- Remove old argument as it was RootWindow's size but not HostWindow's, and it's not used.
BUG=none
TEST=none
Review URL: https://chromiumcodereview.appspot.com/16410015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@205706 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
the launcher.
Everything basically works with this patch, but two essential things are still missing:
1. The icon which gets dragged should get its own widget so that it can get visually dragged outside the app list.
2. The unit tests.
They will be send as a second patch because of:
a. The patch is already pretty big as it is.
b. I want to make get this "signed off" before continuing this route.
BUG=166429
TEST=visual, tests come with second patch
Review URL: https://chromiumcodereview.appspot.com/14533006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@199016 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
> ash: Update app list button.
>
> - Update the icons assets;
> - Make app list button a ToggleImageButton and wire its
> toggle state with app list UI visibility;
> - Make app list button icon vertically center aligned and
> get rid of the special handling;
> - Expose toggled state from ToggleImageButton so that it
> could be checked in test;
>
> BUG=235994
> R=sky@chromium.org
>
> Review URL: https://codereview.chromium.org/13993027
TBR=jamescook@chromium.org
Review URL: https://codereview.chromium.org/14863006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@198478 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Update the icons assets;
- Make app list button a ToggleImageButton and wire its
toggle state with app list UI visibility;
- Make app list button icon vertically center aligned and
get rid of the special handling;
- Expose toggled state from ToggleImageButton so that it
could be checked in test;
BUG=235994
R=sky@chromium.org
Review URL: https://codereview.chromium.org/13993027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@197366 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Rename ArrowLocation and ArrowLocationMask for brevity.
Rename accessors and identifiers to match.
Inline the only use of GetInsetsForArrowLocation.
Rewrite is_arrow_on_[left|top] as one-liners.
Combine previously too-long lines; Minor other cleanup.
BUG=None (cleanup)
TEST=None; unit tests and no bubble regressions.
TBR=ben@chromium.org
Review URL: https://chromiumcodereview.appspot.com/14105005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193615 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When the app menu is open, swiping the launcher to hide it should close
the app menu first. It is not reasonable to show the app menu when there
is no launcher visible.
BUG=170900
Review URL: https://chromiumcodereview.appspot.com/12622010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@188064 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Refactor the classes related with displaying the shelf (background behind the launcher/status area widget) to be a separate class. Removing background delegates from the launcher and status area widget (represented on tray views).
TBR=ben@chromium.org
BUG=163002
Review URL: https://chromiumcodereview.appspot.com/12313118
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@187122 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Now clicking on the shadowy bit will go through to whatever is underneath.
TBR=sky for trivial ash change
BUG=166975, 170073
Review URL: https://chromiumcodereview.appspot.com/12335124
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@185097 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adds a WidgetObserver event that is guaranteed to be invoked synchronously
when a Widget is requested to close. Rename the existing event, which
is invoked at Widget destruction time, typically via a task on the event
loop, to OnWidgetDestroying.
The new event will be used in Views-specific WebContentsModalDialogManager
functionality to get close events for Widgets, so that we don't risk
adverse interleaving of Widget close events with other relevant events
(e.g. IPC). This will allow us to avoids test flakiness observed
in https://codereview.chromium.org/12045037 due to event races in
PrintPreviewTest.PrintCommands
BUG=157161
Review URL: https://chromiumcodereview.appspot.com/12089061
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180796 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
align).
BUG=159770
Review URL: https://chromiumcodereview.appspot.com/11885007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@176556 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
OS=="mac".
Moves ui/app_list code specific to toolkit-views to ui/app_list/views.
Update gyp[i], include guards, #includes for moved files.
Seed chrome/browser/ui/cocoa/app_list and ui/app_list/cocoa with a
skeleton Cocoa UI.
BUG=138633
TEST=Mostly build changes. Functionality of AppLauncher on ChromeOS
and Windows should be unchanged.
On OSX, without chrome running, `Chromium.app/Contents/MacOS/Chromium --show-app-list`
should show a borderless gray window for 1 second (note: this is for
testing only -- OSX needs a different approach for IPC).
Review URL: https://chromiumcodereview.appspot.com/11784034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@176042 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This is the last step. Once this is done, we can possibly hide EventResult.
BUG=163618
Review URL: https://codereview.chromium.org/11592011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173653 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=165419
TEST=on a dev-mode device, change the launcher's auto-hide behavior and see how it's stored in Preferences
Review URL: https://codereview.chromium.org/11570017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173508 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
http://crbug.com/162100
R=sadrul@chromium.org
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=172624
Review URL: https://codereview.chromium.org/11537019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172674 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
http://crbug.com/162100
R=sadrul@chromium.org
Review URL: https://codereview.chromium.org/11537019
TBR=ben@chromium.org
Review URL: https://codereview.chromium.org/11558027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172646 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
http://crbug.com/162100
R=sadrul@chromium.org
Review URL: https://codereview.chromium.org/11537019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172624 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Share the launcher mode/delegate in multiple launchers.
This CL moves the launcher mode/delegate to Shell
- Initialize Display objects before creating RootWindow
for 2nd as it'll be used during creation.
I'll add tests in next CL, as it crashes without https://codereview.chromium.org/11515004/.
BUG=145978
TEST=none
Review URL: https://chromiumcodereview.appspot.com/11465037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172310 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Focus observation @ target -> FocusChangeObserver set on target instead of WindowDelegate methods.
Activation observation @ target -> ActivationChangeObserver set on target instead of ActivationDelegate methods.
Changes FocusChange notifications to be sent regardless of whether or not focus is being cleared.
Changes signature of both observers to carry gaining and losing focus/activation.
http://crbug.com/162100
R=sky@chromium.org
Review URL: https://codereview.chromium.org/11490015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@172258 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Creates and populates AppListModel and starts icon loading for
current page apps before creating views;
- Shows UI when all icons are loaded or a maximum wait time has
passed (currently 50ms);
BUG=161025
TEST=Bring up app launcher and the app icons should not flash.
R=sky@chromium.org
Review URL: https://chromiumcodereview.appspot.com/11418312
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171299 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Cleanup: remove redundant shelf code.
- replaces switch to get values for alignment with template
- eliminate alignment member variables and use shelf_layout_manager where possible.
other minor clean ups: remove unnecessary namespace, use member variable.
BUG=151417
TEST=none
Review URL: https://chromiumcodereview.appspot.com/11434099
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@171120 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The EventHandler overriddes should mark the event as having been processed
(Event::SetHandled) when the event triggered an action, or stop the propagation
of the event (Event::StopPropagation) as appropriate (when the handler destroyed
the event-target, for example).
BUG=163618
Review URL: https://codereview.chromium.org/11280290
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170890 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
http://crbug.com/162100
R=sky@chromium.org
Review URL: https://codereview.chromium.org/11421194
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170752 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This allows us to have multiple focus system implementations.
http://crbug.com/162100
R=sky@chromium.org
Review URL: https://codereview.chromium.org/11299219
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@169824 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This allows getting rid of a whole bunch of noop implementations, and makes
it easier to add new event types to be dispatched through the EventDispatcher
to EventHandlers.
BUG=159632
Review URL: https://codereview.chromium.org/11348145
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@168817 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Add an over scroll animation duration of 50ms;
- Use the over scroll animation duration for transition to invalid page
(i.e. over scroll); This was using normal page transition duration of 180ms
before.
- Use the same duration for snap back animation; This was 100ms and now changed
to 50ms.
Also fixed a bug that app launcher bubble moves to wrong location if starting an
over scroll before previous over scroll snap back animation finishes.
BUG=160590
TEST=Verify over scroll animation is fast enough.
R=sky@chromium.org
Review URL: https://chromiumcodereview.appspot.com/11275320
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167784 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In this CL, the following EventFilters are converted into EventHandlers:
* AcceleratorFilter
* MagnificationControllerImpl
* TrayEventFilter
* TouchObserverHUD
* AppListController
* ShelfLayoutManager::AutoHideEventFilter
* SystemGestureEventFilter
BUG=159632
Review URL: https://codereview.chromium.org/11362196
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167218 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=none
Review URL: https://codereview.chromium.org/11364062
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@165877 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=137285
Review URL: https://chromiumcodereview.appspot.com/11341034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@165221 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- move the code to create/open context menu from launcher to
root window controller.
- updated methods that access shelf/launcher state to take root window so that menu can specify on which display it is working on.
BUG=145978, 155776
TEST=none
Review URL: https://chromiumcodereview.appspot.com/11198078
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@163459 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=156497
TEST=Click/touch right below app list (i.e. in the shadow area) should close it.
R=sky@chromium.org
Review URL: https://chromiumcodereview.appspot.com/11194056
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@162835 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
updated all clients to use one on RootWindowController and/or
Launcher::ForPrimaryDisplay/ForWindow.
BUG=145978
Review URL: https://chromiumcodereview.appspot.com/11017079
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@162664 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=150656
TEST=Verify app list bubble is at the right location on all desktops.
R=oshima@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10996066
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@159804 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is because cached bubble bounds |view_bounds_| at bubble creation time
might not be correct when launcher is hidden and the bubble moves when launcher
shows up. The fix is to defer getting the bubble location until first
over-scroll movement.
BUG=150851
TEST=Verify app list bubble at expected location when over-scrolling with hidden launcher.
R=sky@chromium.org
Review URL: https://codereview.chromium.org/10967021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@158007 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Per 135200, we should only use image based code.
- Add two new sets of bubble border images;
- Use the new bubble border images for app menu and launcher tooltip;
- Get rid of BubbleBorder2 since it's no longer needed;
BUG=135200
TEST=Verify app list, launcher bar tooltip bubbles use shadow based on image and no artifacts.
R=msw@chromium.org,sky@chromium.org,oshima@chromium.org
Review URL: https://codereview.chromium.org/10905311
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@158003 0039d316-1c4b-4281-b951-d872f2087c98
|