| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
| |
Nothing (much) renders. WIP.
http://crbug.com/93947
TEST=none
Review URL: http://codereview.chromium.org/7849004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100773 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
There is a chance that window is no longer valid and the call to XGetWindowAttributes will fail and leave |attributes| uninitialized.
BUG=chromium-os:16986
TEST=manual
Review URL: http://codereview.chromium.org/7831063
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100772 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previously, it was possible for NetworkChangeNotifierMac to unschedule
the SCNetworkReachability callback before it had been scheduled. Under the
hypothesis that this is causing crashes on shutdown, this change makes sure
that the helper thread (on which reachability is initially scheduled) runs to
completion before we check to see if we need to unschedule the callback.
BUG=91767
Review URL: http://codereview.chromium.org/7862006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100771 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Appease the clang bot.
BUG=chromium-os:19908
TEST=clang bot succeeds
TBR=dmazzoni@chromium.org
Review URL: http://codereview.chromium.org/7845027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100770 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=96312,96313
TEST=none
TBR=jhawkins
Review URL: http://codereview.chromium.org/7877001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100769 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
necessary for maximize.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/7748047
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100768 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
targets are very simple and used little code from chrome targets. However their dependency on chrome targets was problematic because a lot of code wasn't being built for 64 bit on Windows, and so there were a lot of "dummy" files being added with stub functions and code was also being compiled out in random places for NACL_WIN64.
I've made the NaCl 64 bit windows targets self contained. They do use a few files from common, but those files are self-contained. In the future, we could move these to be in the same 64 bit target as the constants from common. However that won't make a maintenance difference since someone could still introduce link dependencies to other files in common.
Additionally, since we're not using chrome code anymore, we can avoid having both nacl.exe and nacl.dll. nacl.exe is sufficient, and this saves 1.4MB of uncompresed binaries in the installer.
BUG=86322
Review URL: http://codereview.chromium.org/7863024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100767 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=88600
TEST=Unittests
Review URL: http://codereview.chromium.org/7867019
TBR=sergeyu@chromium.org
Review URL: http://codereview.chromium.org/7880002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100766 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
channel_ has been reset by OnChannelError after a round-trip.
BUG=chromium-os:18390
TEST=Open a new tab on cros. Close all other tabs. Navigate to google.com. Close the tab. Should no longer see the sad tab.
Review URL: http://codereview.chromium.org/7864019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100765 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
+ Fixes clang issue
Original CL: http://codereview.chromium.org/7828064/
BUG=chromium-os:19908, chromium-os:19252
TEST=Test UI connectint to a VPN
Review URL: http://codereview.chromium.org/7870011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100764 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=gyp runs without error
TBR=lipalani
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100763 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
ScriptShape was incorrectly handling USP_E_SCRIPT_NOT_IN_FONT.
The previous code would hang on unshapable text.
For now, unshapable glyphs are rendered as missing (⛿).
I added a TODO regarding the need for actual font fallback.
BUG=90426
TEST=No hang on hebrew text, etc.
Review URL: http://codereview.chromium.org/7863010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100762 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
These entries were added at revision r51409, but they were moved to ui/gfx/
at revision r73530.
R=sky@chromium.org,evan@chromium.org
Review URL: http://codereview.chromium.org/7873008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100761 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If the text entered into the password field has 16 lowercase alphabetical characters, or is 4 clusters of 4 lowercase alphabetical characters separated by spaces, add a second confirmation step to reduce the chance users encrypt their data with an ASP.
BUG=none
TEST=enter an ASP into the password field on sync signin, notice warning message.
Review URL: http://codereview.chromium.org/7859011
TBR=tim@chromium.org
Review URL: http://codereview.chromium.org/7877003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100760 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fix for data race in WebDataService::WebDataRequest::Cancel. Also removes the suppression. This was happening with very low frequency. Probably during shutdown. It appears to have been largely benign because the lock within WebDataService::CancelRequest would protect the important structures. This change adds a lock to the WebDataRequest::Cancel() operation and makes the query WebDataRequest::IsCancelled() atomic with respect to the consumer and the cancelled state.
BUG=40244
TEST=Run TSAN on bots, see that they stay green.
Review URL: http://codereview.chromium.org/7870007
TBR=dhollowa@chromium.org
Review URL: http://codereview.chromium.org/7879001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100759 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=None
Review URL: http://codereview.chromium.org/7780015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100758 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Fix for data race in WebDataService::WebDataRequest::Cancel. Also removes the suppression. This was happening with very low frequency. Probably during shutdown. It appears to have been largely benign because the lock within WebDataService::CancelRequest would protect the important structures. This change adds a lock to the WebDataRequest::Cancel() operation and makes the query WebDataRequest::IsCancelled() atomic with respect to the consumer and the cancelled state.
BUG=40244
TEST=Run TSAN on bots, see that they stay green.
Review URL: http://codereview.chromium.org/7870007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100757 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=chromium-os:18413
R=mkrebs@chromium.org
TEST=
Review URL: http://codereview.chromium.org/7780018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100756 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
native client repo to the Chrome one. The Native Client Authors are currently a
subset of the Chromium Authors.
TEST=none
BUG=none
Review URL: http://codereview.chromium.org/7778046
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100755 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
for Pepper interfaces, and replaces it with a list of macros. When files want to know which Pepper interface names and structs there are, they define what they want to do with the macros, and then include the relevant files for the classes of interfaces they want (stable, private, dev).
This does not convert all the dev interfaces. I just did a few to keep the patch smaller. So there is still a lot of manual registration.
This fixes the previous design problem where we assumed one *_Proxy object == one interface. We have been hacking around this lately with duplicate GetInfo calls, but this doesn't work for PPP interfaces.
Now, a _Proxy object is just there to help keep things organized. One proxy can handle zero, one, or many interfaces, and this mapping is controlled by just one line in the interfaces file.
So for example, to add a new function to a new version of an interface with backward compatibility, you would add that function to the _api.h file, and write a thunk for the new interface. Then you only need to add one line to the interfaces_ppb_public_stable.h file and that will be hooked up with the proxy and the implementation.
This removes some _proxy objects/files that were used only to declare that the interfaces existed, since they're no longer necessary.
I folded Console into the Instance API which removed a bunch of code.
I removed FileChooser 0.4. I think everybody has converted to the new one, and I think parts of it weren't even hooked up properly anymore.
Review URL: http://codereview.chromium.org/7740038
TBR=brettw@chromium.org
Review URL: http://codereview.chromium.org/7844018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100754 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TEST=it compiles
Review URL: http://codereview.chromium.org/7871005
TBR=brettw@chromium.org
Review URL: http://codereview.chromium.org/7872007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100752 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
TEST=it compiles
Review URL: http://codereview.chromium.org/7871005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100751 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Revert task manager test. This one looks to be more complicated.
See http://codereview.chromium.org/7745037/ (original review)
R=phajdan.jr@chromium.org
BUG=None.
TEST=browser_tests,ui_tests
Review URL: http://codereview.chromium.org/7831020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100749 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
interfaces, and replaces it with a list of macros. When files want to know which Pepper interface names and structs there are, they define what they want to do with the macros, and then include the relevant files for the classes of interfaces they want (stable, private, dev).
This does not convert all the dev interfaces. I just did a few to keep the patch smaller. So there is still a lot of manual registration.
This fixes the previous design problem where we assumed one *_Proxy object == one interface. We have been hacking around this lately with duplicate GetInfo calls, but this doesn't work for PPP interfaces.
Now, a _Proxy object is just there to help keep things organized. One proxy can handle zero, one, or many interfaces, and this mapping is controlled by just one line in the interfaces file.
So for example, to add a new function to a new version of an interface with backward compatibility, you would add that function to the _api.h file, and write a thunk for the new interface. Then you only need to add one line to the interfaces_ppb_public_stable.h file and that will be hooked up with the proxy and the implementation.
This removes some _proxy objects/files that were used only to declare that the interfaces existed, since they're no longer necessary.
I folded Console into the Instance API which removed a bunch of code.
I removed FileChooser 0.4. I think everybody has converted to the new one, and I think parts of it weren't even hooked up properly anymore.
Review URL: http://codereview.chromium.org/7740038
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100748 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
since the code path does not exist anymore.
BUG=52371
TEST=none
Review URL: http://codereview.chromium.org/7863021
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100747 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/7826034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100746 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
The leak from IBusController was fixed in crrev.com/100723
TEST=the leak is now gone
BUG=87997
Review URL: http://codereview.chromium.org/7748046
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100745 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
DefaultClientSocketFactory::CreateTCPClientSocket() hasn't existed since 80781 where it was renamed to CreateTransportClientSocket().
BUG=22450
TEST=none
Review URL: http://codereview.chromium.org/7872003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100744 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=No code change
Review URL: http://codereview.chromium.org/7847024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100743 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Add proper config for Android to libevent. Enable libevent to compile without needing __linux__.
BUG=
TEST=
Review URL: http://codereview.chromium.org/7866018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100742 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=96295
TEST=none
TBR=jhawkins
Review URL: http://codereview.chromium.org/7867050
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100741 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
The XIB changes involve unifying the 'connected' message and username field into a single text field and adding some links to we can access the button in order to set its text.
BUG=93088
TEST=Manual
Review URL: http://codereview.chromium.org/7796024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100740 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Chrome fails at RAW_CHECK(g_shutdown_pipe_write_fd != -1) if it receives signal in a window
where signal handlers are registered, but shutdown pipe isn't setup yet.
BUG=chromium-os:17606
TEST=autotest with session manager sending SIGTERM no longer crash
Review URL: http://codereview.chromium.org/7464013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100739 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
suppression.
BUG=53989,96298,96300
TEST=none
TBR=jhawkins
Review URL: http://codereview.chromium.org/7844015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100738 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
SyncSetupHandler, which also handles the UI for ClientLogin and for other parts of Sync setup.
It leaves the non-UI elements, such as token exchange and user info, in SigninManager, which also handles similar tasks for the ClientLogin flow.
BUG=95803
TEST=manual
Review URL: http://codereview.chromium.org/7848016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100737 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
It wasn't clear to me in some codepaths whether these functions were
being called on the correct thread. This assertion will help catch
errors.
Review URL: http://codereview.chromium.org/7867011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100736 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=96296
TEST=ExtensionServiceTest.UninstallExtension
Review URL: http://codereview.chromium.org/7844013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100735 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=chromium-os:20268
TEST=tests and autotests succeed
Review URL: http://codereview.chromium.org/7863011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100734 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Logic:
- Profile shutdown kills all renderer processes
- When the RMF detects the channel closing, it kills off any plugin channel requests from:
* PluginService (which needs to check the PluginServiceFilter on the FILE thread)
* PluginProcessHost (which may need to wait for the channel to get established)
- RMF uses the new OnPluginProcessHostFound() to cancel at the PluginService or PluginProcessHost
BUG=94704
TEST=New PluginService test, no known manual test
Review URL: http://codereview.chromium.org/7867031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100733 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
R=sky@chromium.org
TEST=unit_tests --gtest_filter=TabMenuModelTest*
Review URL: http://codereview.chromium.org/7844004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100732 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=96295
TEST=none
TBR=jhawkins
Review URL: http://codereview.chromium.org/7844011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100731 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
This is important for shutdown ordering to make sure the objects hanging off the channel get deleted before objects they reference (like data in ResourceContext) go away.
BUG=91398
TEST=none
Review URL: http://codereview.chromium.org/7796029
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100730 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
read-only check-out)
BUG=94920
-Related:
Check in a new CRX:
http://codereview.chromium.org/7828014/
Check in a few GYP changes:
http://codereview.chromium.org/7827021/
-Modified profile_impl to search for and auto-install apps found in DIR_DEFAULT_APPS when profile created
-Added tracking code for number of extension installs and uninstalls and number of webstore visits.
Review URL: http://codereview.chromium.org/7839009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100727 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/7863009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100726 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
handled by ChromeOS
the balloon notification class which is a RVHD. We now handle this message in RVH and call out
the new virtual function in WebUISend in the RVHD interface which the balloon class overrides.
Continuing changes to get rid of the pattern of IPC messages spanning across
content and chrome.
BUG=87335
TEST=No change in functionality
Review URL: http://codereview.chromium.org/7864016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100725 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=94968
TEST=Visual
Review URL: http://codereview.chromium.org/7867045
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100724 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
IBusController used to be a singleton, but this change lets
IBusController be owned by InputMethodManager.
FWIW, here's the command line to run a test in the browser_tests:
% sh tools/valgrind/chrome_tests.sh --build_dir=out/Release -t browser --gtest_filter=ExtensionApiTest.InputMethodApiBasic
BUG=chromium:86146
TEST=ran a browser test locally and the leak is gone after the change. Alos tested on the device to confirm that input methods worked as before.
Review URL: http://codereview.chromium.org/7864023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100723 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Retry of r100550 with the gypi file fixed.
Review URL: http://codereview.chromium.org/7796028
TBR=dfilimon@google.com
Review URL: http://codereview.chromium.org/7754027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100722 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1. never resize the web store icon when showing the promo
2. don't pulse anything other than app-contents (the icon + span). This includes for example apps-promo-extras and the app notification)
3. don't pulse when the user clicks inside app/app-contents, but outside the clickable region
BUG=94783
TEST=see above
Review URL: http://codereview.chromium.org/7860013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100721 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
this no longer reports leak.
TBR=jochen@chromium.org
BUG=90963
TEST=none
Review URL: http://codereview.chromium.org/7844005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100717 0039d316-1c4b-4281-b951-d872f2087c98
|