| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
[Chrome]HistogramTester.
Design Doc (Googlers Only):
https://docs.google.com/a/google.com/document/d/1RUY0BcxBppdkwFP3T8qbQmQStBRFGsLmuR4WuIXhiCA/edit
BUG=384011
TBR=timsteele@chromium.org
Review URL: https://codereview.chromium.org/379283002
Cr-Commit-Position: refs/heads/master@{#291207}
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291207 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
The state is now taken from the label with the instructions within the bookmark bar.
Now an empty bookmark bar's label is accessible.
BUG=395895
Review URL: https://codereview.chromium.org/408103002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@285704 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=160586
Review URL: https://codereview.chromium.org/399493002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283659 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change just exposes information about what tabs are selected
and fires proper notifications when tab selection changes. This will
allow screen readers to announce when switching tabs even if the
web content doesn't have focus. It's also a good first step towards
making the tab strip fully accessible too (e.g. making it possible to
perform operations on multiple tabs with just the keyboard).
As part of this change I renamed the "selection changed" event to
"text selection changed" so that there's no confusion between the
text-selection-related events, and tab/list selection events.
BUG=100412
Review URL: https://codereview.chromium.org/349033010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@282396 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is used for touch accessibility on Chrome OS, see bug. Fires the
AX event on mouse enter and passes this through to the
accessibilityPrivate API for ChromeVox.
BUG=377043
Review URL: https://codereview.chromium.org/297133002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@273529 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If the Broken Window Theory holds true, then ExtensionService will never be
cleaned up (and will, in fact, degrade faster than we can fix it) if it looks
like it's messy. So, let's clean house!
- Group overriden methods together, say what they override.
- Move all comments to the original method declaration (i.e., if it's overriden
from ExtensionServiceInterface, comment there).
- Move accessors together.
- Move for_test methods together.
- Move methods which are not used publicly into private:.
- Move simple getters/setters from the .cc to the .h.
- Remove FRIEND_TESTs that should not be there.
- Remove unneeded forward class declarations.
- Remove unneeded includes.*
*This one led to a BUNCH of fixes in other files, which relied on the
transitive includes of extension_service.h. Sorry for the noise.
BUG=351891
TBR=jochen@chromium.org (for every header file change - they're only fixing transitive includes)
Review URL: https://codereview.chromium.org/296073003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@273424 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Changes:
1- Change it to use ui_strings.h
2- Use IDS_APP_OK and IDS_APP_CANCEL from ui_strings.
3- Move to components/infobars/core from chrome/browser/infobars (done
with tools/git/move_source_file.py).
BUG=373234
TEST=chrome still links and run fine.
R=droger@chromium.org, pkasting@chromium.org
TBR=ben # for trivial chrome/ changes
Review URL: https://codereview.chromium.org/289083004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@272036 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The reason for doing this is that user_prefs depends on content_browser,
which brings a lot of extra dependencies that takes a lot of time to build
(yeah, I'm looking at you Blink and V8) and bookmarks is one of the targets
that should not depend on content.
BUG=359565
TEST=green trybots
R=blundell@chromium.org, sky@chromium.org, yoz@chromium.org
Review URL: https://codereview.chromium.org/271793003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@270722 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The only difference from the original change is a fix to manifest_guest.json
to prevent ChromeVox from breaking in guest mode. (I also added a new test to
cover that scenario and prevent future regressions.)
Original code review: https://codereview.chromium.org/106973003
BUG=278540,362442
TBR=thakis
Review URL: https://codereview.chromium.org/268823002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@269243 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
function doesn't require divergent return value semantics depending on subclass.
BUG=365732
R=rockot@chromium.org
TBR=pkasting@chromium.org,dtseng@chromium.org
Review URL: https://codereview.chromium.org/252653002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@267083 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit 5d76e14014f504a496e111ecf23a433f1bbd5897.
Original review:
https://codereview.chromium.org/106973003
BUG=362442,278540
TBR=dmazzoni
TEST=Manually verify ChromeVox starts on guest mode.
NOTRY=true
Review URL: https://codereview.chromium.org/258063008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@266737 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/246403002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@265580 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL sets up the directory structure for the Infobars component and the
infobars_core target. It also does a simple code move to the compoonent.
This CL does not move ConfirmInfoBarDelegate because it still has a
dependency on chrome/ (for strings).
BUG=354379
TBR=sky, reed
Review URL: https://codereview.chromium.org/240193003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@264792 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=362672
Review URL: https://codereview.chromium.org/239453011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@264478 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This allows to change the implementation of InfoBarDelegate::web_contents()
such that it no longer calls InfoBarManager::web_contents() (which will be
removed), but InfoBarService::web_contents() instead.
BUG=354379
TBR=jochen
Review URL: https://codereview.chromium.org/228293004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@262976 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
We can land this after cl/57741007 is landed and rolled
into Chrome first.
BUG=278540
Review URL: https://codereview.chromium.org/106973003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@261291 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
Found by Scythe.
Review URL: https://codereview.chromium.org/217323007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@261030 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
InfoBarService is now responsible for associating an InfoBarManager to a Tab
and handling the interactions with the rest of chrome (notifications, navigation
events).
InfoBarManager has the core logic for infobar management. It owns the list of infobars.
Eventually InfoBarManager will no longer rely on content/, but for now it still
has a reference to a WebContents instance. This reference was kept for now to
minimize the size of the change, but will be removed in a future CL.
BUG=354379
TBR=jochen
Review URL: https://codereview.chromium.org/211273007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@260847 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
We've seen different failure related to the fact that we use the same flag without providing the same API and functionality.
BUG=
TBR=jamesr
Review URL: https://codereview.chromium.org/212643015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@259855 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This breaks another src/extensions -> src/chrome dependency.
* Mechanical file move.
* Clean up DEPS.
* Move pinch-to-zoom handling to ExtensionViewHost as it only applies to
visible extension content.
No functional changes.
BUG=321341
TEST=browser_tests *Extension*
TBR=sky@chromium.org for header file rename across chrome/browser
Review URL: https://codereview.chromium.org/198703006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@258494 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Gets rid of the enums defined in accessibility_types.h and switches to the
enums defined in ui/accessibility/ax_enums.h, which are shared by "content".
Eventually, content and views may share more common accessibility code, but
moving to the same enums is a good first step.
BUG=316726
R=piman@chromium.org, sky@chromium.org, tfarina@chromium.org
Review URL: https://codereview.chromium.org/126763003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@253207 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This simplifies the unload codepath for ChromeVox.
- remove event listener observer
- remove loaded ChromeVox profile tracking
- send events to ChromeVox for load using lazy dispatch
TEST=Manual. Verify load speaks "ChromeVox, spoken feedback enabled" the first time only.
BUG=339775,332461
Review URL: https://codereview.chromium.org/151093003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@248439 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This moves ExtensionSystem to extensions/browser.
It also moves setting the channel for Features to ChromeExtensionsBrowserClient.
It also removes the duplicate ExtensionSystem::GetForBrowserContext.
BUG=337707
TBR=sky@chromium.org
Review URL: https://codereview.chromium.org/147923005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247788 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
namespace.
BUG=330556
TEST=no change
TBR=ben@chromium.org
Review URL: https://codereview.chromium.org/109013006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@242447 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=88666
TEST=no change
R=brettw@chromium.org
TBR=ben@chromium.org
Review URL: https://codereview.chromium.org/106433007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@242387 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This changes the ownership model of infobars so that InfoBars are long-lived and
own their InfoBarDelegates directly. The InfoBarService pseudo-owns the
InfoBars (instead of deleting them directly, it tells them when they're unowned
and expects them to delete themselves).
This fixes leaks when infobars are closed while not visible (e.g. in a
background tab) and in general makes the system clearer and easier to reason
about.
BUG=62154
TEST=none
TBR=sky
Review URL: https://codereview.chromium.org/103993003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@239110 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This updates portions of chrome/browser/... to use the base namespace.
BUG=
Review URL: https://codereview.chromium.org/105193002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238766 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Crashes on android
BUG=325216
> Infobar system refactor.
>
> This changes the ownership model of infobars so that InfoBars are long-lived and
> own their InfoBarDelegates directly. The InfoBarService pseudo-owns the
> InfoBars (instead of deleting them directly, it tells them when they're unowned
> and expects them to delete themselves).
>
> This fixes leaks when infobars are closed while not visible (e.g. in a
> background tab) and in general makes the system clearer and easier to reason
> about.
>
> BUG=62154
> TEST=none
> R=erg@chromium.org, sail@chromium.org, sky@chromium.org
>
> Review URL: https://codereview.chromium.org/22694006
TBR=pkasting@chromium.org
Review URL: https://codereview.chromium.org/102163002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238402 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This changes the ownership model of infobars so that InfoBars are long-lived and
own their InfoBarDelegates directly. The InfoBarService pseudo-owns the
InfoBars (instead of deleting them directly, it tells them when they're unowned
and expects them to delete themselves).
This fixes leaks when infobars are closed while not visible (e.g. in a
background tab) and in general makes the system clearer and easier to reason
about.
BUG=62154
TEST=none
R=erg@chromium.org, sail@chromium.org, sky@chromium.org
Review URL: https://codereview.chromium.org/22694006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238283 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=322729
TEST=manual with ChromeVox
Review URL: https://codereview.chromium.org/79543010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@238047 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Now that its dependencies on Profile, ActivityLog, RuntimeAPI and
WebRequestAPI have been removed it can move into the extensions module.
This is part of the AppShell extensions refactoring project.
BUG=316367
TEST=compiles
R=yoz@chromium.org
TBR=mukai@chromium.org for mechanical file move in c/b/notifications, dmazzoni@chromium.org for mechanical file move in accessibility code
Review URL: https://codereview.chromium.org/78483003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@236549 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This makes battery alerts and network state change notifications emit
spoken (and braille) feedback again and fixes a crash on ash debug
builds when accessibility is enabled.
BUG=310766
R=dmazzoni@chromium.org
Review URL: https://codereview.chromium.org/74083003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@235764 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This fix is not meant to redo the API, but simply to make it so ChromeVox can verbalize this important control. We should note the issues with this API in terms of indicating the caller's intent and some underlying assumptions made about what events can be fired on what roles.
BUG=318463
TEST=Ensure that the "apps" wrench menu gets announced by ChromeVox.
Review URL: https://codereview.chromium.org/71863003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@234949 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
AccessibilityManager::Speak and AccessibilityManager::MaybeSpeak.
BUG=316710
TEST=Sign in and observe output in both braille and speech.
Review URL: https://codereview.chromium.org/66583007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@234876 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=117261
TBR=dmazzoni@chromium.org (accessibility)
TBR=jam@chromium.org (automation)
TBR=pkasting@chromium.org (infobars, ui/views)
TBR=thakis@chromium.org (ui/cocoa)
TBR=erg@chromium.org (ui/gtk)
TBR=jennb@chromium.org (ui/panels)
No functional changes, just a namespace move
Review URL: https://codereview.chromium.org/23618040
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@234715 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is part of a refactor to extract extensions code from Chrome.
To allow the move:
* Extract extensions::file_util from extension_file_util.h/cc
* Move switches::kAllowHTTPBackgroundPage to extensions/common/switches.h
BUG=316218
TEST=compiles, unit_tests
TBR=ben@chromium.org for mechanical header file renames across chrome/browser
Review URL: https://codereview.chromium.org/65123002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@234065 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
ts to occur directly in the extension.
BUG=261451
TEST=On Aura, rapidly turn ChromeVox on and off (on login/locked screen, user s
creens). Verify behaviors in above bug are true. Depends on a ChromeVox change.
Review URL: https://codereview.chromium.org/42743003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@232886 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This instead creates new variants of ExtensionFunction for Chrome APIs,
which need Profile, and uses them for any API that needs to access
Profiles.
TBR=sky@chromium.org
BUG=297942
Review URL: https://codereview.chromium.org/35893010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@231406 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
generated event names instead of custom constant strings.
BUG=268380
Review URL: https://chromiumcodereview.appspot.com/23013006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218378 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=darin@chromium.org
BUG=268984
TEST= no change
Review URL: https://chromiumcodereview.appspot.com/22922003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218171 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Remove it from a lot of API tests that don't need it anymore.
BUG=162530
TBR=sky@chromium.org
Review URL: https://chromiumcodereview.appspot.com/20593003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@214836 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
version.
The original intent by disabling them per OS version was that regular browser tests don't run on win8_aura bots anyways... but this also makes it annoying as devs on Win8+Aura now can't run these tests as "regular" browser tests...
Disable those tests by detecting the --ash-browsertests command-line flag instead.
BUG=262796
Review URL: https://chromiumcodereview.appspot.com/21112003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@214488 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
--ash-browsertests)
Ran the tests locally and disabled all failing tests (I think...), will run a second pass locally and then we can push them on the waterfall
Temporarily disabled tests are tracked @ https://docs.google.com/a/google.com/document/d/110YdKCAj3MxECZcW3iwYLjgTdzs2aZicLeA8OrC20Zo/edit
BUG=179830, 262796
Review URL: https://chromiumcodereview.appspot.com/19697010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212940 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TBR=brettw
Review URL: https://codereview.chromium.org/18640003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@211261 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Shorten code
* Eliminate using directives
* Rename "infobar" to "delegate" in a few places for clarity
* Add a few DCHECKs
BUG=none
TEST=none
R=dmazzoni@chromium.org, isherman@chromium.org, jam@chromium.org
Review URL: https://codereview.chromium.org/18209021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@211254 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
TBR=ben@chromium.org
Review URL: https://chromiumcodereview.appspot.com/15789014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@205122 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Splits apart the GetIcon() call to allow accessing the ID of the resource in
addition to the loaded image. This is needed so that the Android port can
determine which of the Java-side resources it needs to load for its UI.
* GetIconID() is a function that gets overridden by subclasses instead of
GetIcon(). GetIconID() returns the IDR_* tag that represents the icon that
it would normally load.
* GetIcon() now calls GetIconID() and shouldn't be overridden.
* Small change to the SimpleAlertInfoBarDelegate::Create() call to account
for invalid icon IDs (was NULL for bitmaps, 0 for non-existent icons).
BUG=237034
Review URL: https://chromiumcodereview.appspot.com/15067008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@200598 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=180785
R=joi@chromium.org
TBR=darin@chromium.org
Review URL: https://chromiumcodereview.appspot.com/14622003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@198384 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=
Review URL: https://chromiumcodereview.appspot.com/14473011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@198196 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Don't define complex functions inline
* Use DISALLOW_COPY_AND_ASSIGN where appropriate
* Make consistent the headers on sections of a class declaration or .cc file
* Use two newlines between major sections of a .cc file
BUG=none
TEST=none
R=sky@chromium.org
Review URL: https://codereview.chromium.org/14619002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@198027 0039d316-1c4b-4281-b951-d872f2087c98
|